From 5d01d2d16f4a4c9592002d7c76c19f01f2529140 Mon Sep 17 00:00:00 2001 From: Carol Eidt Date: Tue, 3 Jul 2018 09:46:04 -0700 Subject: [PATCH] Remove Interval::isMultiReg This is no longer used after #16517 --- src/jit/lsra.cpp | 19 +------------------ src/jit/lsra.h | 5 ----- 2 files changed, 1 insertion(+), 23 deletions(-) diff --git a/src/jit/lsra.cpp b/src/jit/lsra.cpp index 01192b2..654b340 100644 --- a/src/jit/lsra.cpp +++ b/src/jit/lsra.cpp @@ -2684,8 +2684,7 @@ regNumber LinearScan::tryAllocateFreeReg(Interval* currentInterval, RefPosition* { // Don't use the relatedInterval for preferencing if its next reference is not a new definition, // or if it is only related because they are multi-reg targets of the same node. - if (!RefTypeIsDef(nextRelatedRefPosition->refType) || - isMultiRegRelated(nextRelatedRefPosition, refPosition->nodeLocation)) + if (!RefTypeIsDef(nextRelatedRefPosition->refType)) { relatedInterval = nullptr; } @@ -4919,18 +4918,6 @@ bool LinearScan::registerIsFree(regNumber regNum, RegisterType regType) return isFree; } -// isMultiRegRelated: is this RefPosition defining part of a multi-reg value -// at the given location? -// -bool LinearScan::isMultiRegRelated(RefPosition* refPosition, LsraLocation location) -{ -#ifdef FEATURE_MULTIREG_ARGS_OR_RET - return ((refPosition->nodeLocation == location) && refPosition->getInterval()->isMultiReg); -#else - return false; -#endif -} - //------------------------------------------------------------------------ // LinearScan::freeRegister: Make a register available for use // @@ -8596,10 +8583,6 @@ void Interval::dump() { printf(" (constant)"); } - if (isMultiReg) - { - printf(" (multireg)"); - } printf(" RefPositions {"); for (RefPosition* refPosition = this->firstRefPosition; refPosition != nullptr; diff --git a/src/jit/lsra.h b/src/jit/lsra.h index e646294..f88f2d9 100644 --- a/src/jit/lsra.h +++ b/src/jit/lsra.h @@ -1056,7 +1056,6 @@ private: regMaskTP allSIMDRegs(); regMaskTP internalFloatRegCandidates(); - bool isMultiRegRelated(RefPosition* refPosition, LsraLocation location); bool registerIsFree(regNumber regNum, RegisterType regType); bool registerIsAvailable(RegRecord* physRegRecord, LsraLocation currentLoc, @@ -1655,7 +1654,6 @@ public: , isSpecialPutArg(false) , preferCalleeSave(false) , isConstant(false) - , isMultiReg(false) , physReg(REG_COUNT) #ifdef DEBUG , intervalIndex(0) @@ -1726,9 +1724,6 @@ public: // able to reuse a constant that's already in a register. bool isConstant : 1; - // True if this Interval is defined by a node that produces multiple registers. - bool isMultiReg : 1; - // The register to which it is currently assigned. regNumber physReg; -- 2.7.4