Check the "isSafeToMove" predicate, which has a series of tests to make sure
authorBill Wendling <isanbard@gmail.com>
Wed, 28 May 2008 22:52:47 +0000 (22:52 +0000)
committerBill Wendling <isanbard@gmail.com>
Wed, 28 May 2008 22:52:47 +0000 (22:52 +0000)
that it's safe to remat an instruction.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@51659 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/TwoAddressInstructionPass.cpp

index 145f80ae74b2f923f2661508ea6badc2af30b8bb..5f0a923f996e79160cde4a7c1b2be10091bb2b6e 100644 (file)
@@ -330,8 +330,10 @@ bool TwoAddressInstructionPass::runOnMachineFunction(MachineFunction &MF) {
         InstructionRearranged:
           const TargetRegisterClass* rc = MF.getRegInfo().getRegClass(regA);
           MachineInstr *Orig = MRI->getVRegDef(regB);
+          bool SawStore = false;
 
-          if (EnableReMat && Orig && TII->isTriviallyReMaterializable(Orig)) {
+          if (EnableReMat && Orig && Orig->isSafeToMove(TII, SawStore) &&
+              TII->isTriviallyReMaterializable(Orig)) {
             TII->reMaterialize(*mbbi, mi, regA, Orig);
             ReMattedInstrs.insert(Orig);
           } else {