Merge pull request #16727 from CarolEidt/NoIndirByte

Don't force byte regs for indir address
dotnet · Mar 5, 2018 · aea9340 · aea9340
2 parents 5763bb4 + 1765a4e
commit aea9340
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/jit/lsraxarch.cpp b/src/jit/lsraxarch.cpp
@@ -754,7 +754,9 @@ void LinearScan::BuildCheckByteable(GenTree* tree)
         if (tree->OperIsSimple())
         {
             GenTree* op = tree->gtOp.gtOp1;
-            if (op != nullptr)
+            // We need byte registers on the operands of most simple operators that produce a byte result.
+            // However, indirections are simple operators but do not require their address in a byte register.
+            if ((op != nullptr) && !tree->OperIsIndir())
             {
                 // No need to set src candidates on a contained child operand.
                 if (!op->isContained())