[PATCH] D39173: [MachineCSE] Remove redudant TLS access instructions.

Kit Barton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 23 06:46:22 PDT 2017


kbarton added inline comments.


================
Comment at: lib/CodeGen/MachineCSE.cpp:255
+    if (!MRI->isConstantPhysReg(Reg) &&
+        !(TRI->isCallerPreservedPhysReg(Reg, *MI->getParent()->getParent())))
       for (MCRegAliasIterator AI(Reg, TRI, true); AI.isValid(); ++AI)
----------------
Is it not possible to put this check into isConstantPhysReg instead? 



https://reviews.llvm.org/D39173





More information about the llvm-commits mailing list