Remove unused TreeNodeInfo::isHelperCallWithKills field (#11259)
authorBruce Forstall <brucefo@microsoft.com>
Thu, 27 Apr 2017 19:41:43 +0000 (12:41 -0700)
committerGitHub <noreply@github.com>
Thu, 27 Apr 2017 19:41:43 +0000 (12:41 -0700)
Remove unused TreeNodeInfo::isHelperCallWithKills field

src/jit/lsra.cpp
src/jit/nodeinfo.h

index 3718ddf..d558bcc 100644 (file)
@@ -2747,16 +2747,6 @@ regMaskTP LinearScan::getKillSetForNode(GenTree* tree)
         }
         break;
 
-        case GT_LSH:
-        case GT_RSH:
-        case GT_RSZ:
-        case GT_ROL:
-        case GT_ROR:
-            if (tree->gtLsraInfo.isHelperCallWithKills)
-            {
-                killMask = RBM_CALLEE_TRASH;
-            }
-            break;
         case GT_RETURNTRAP:
             killMask = compiler->compHelperCallKillSet(CORINFO_HELP_STOP_FOR_GC);
             break;
@@ -9943,12 +9933,11 @@ void TreeNodeInfo::Initialize(LinearScan* lsra, GenTree* node, LsraLocation loca
         dstCandidates = genRegMask(node->gtRegNum);
     }
 
-    internalIntCount      = 0;
-    internalFloatCount    = 0;
-    isLocalDefUse         = false;
-    isHelperCallWithKills = false;
-    isLsraAdded           = false;
-    definesAnyRegisters   = false;
+    internalIntCount    = 0;
+    internalFloatCount  = 0;
+    isLocalDefUse       = false;
+    isLsraAdded         = false;
+    definesAnyRegisters = false;
 
     setDstCandidates(lsra, dstCandidates);
     srcCandsIndex = dstCandsIndex;
@@ -10373,10 +10362,6 @@ void TreeNodeInfo::dump(LinearScan* lsra)
     {
         printf(" I");
     }
-    if (isHelperCallWithKills)
-    {
-        printf(" H");
-    }
     if (isLsraAdded)
     {
         printf(" A");
index 1937cc4..5f03da2 100644 (file)
@@ -25,7 +25,6 @@ public:
         dstCandsIndex          = 0;
         internalCandsIndex     = 0;
         isLocalDefUse          = false;
-        isHelperCallWithKills  = false;
         isLsraAdded            = false;
         isDelayFree            = false;
         hasDelayFreeSrc        = false;
@@ -117,9 +116,6 @@ public:
     // nodes, or top-level nodes that are non-void.
     unsigned char isLocalDefUse : 1;
 
-    // isHelperCallWithKills is set when this is a helper call that kills more than just its in/out regs.
-    unsigned char isHelperCallWithKills : 1;
-
     // Is this node added by LSRA, e.g. as a resolution or copy/reload move.
     unsigned char isLsraAdded : 1;