[PATCH] D27315: [ARM] Fix for 64-bit CAS expansion on ARM32 with -O0

Tim Northover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 1 13:00:59 PST 2016


t.p.northover accepted this revision.
t.p.northover added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lib/Target/ARM/ARMExpandPseudoInsts.cpp:938
 
-  unsigned SBCrr = IsThumb ? ARM::t2SBCrr : ARM::SBCrr;
-  MIB = BuildMI(LoadCmpBB, DL, TII->get(SBCrr))
-            .addReg(StatusReg, RegState::Define | RegState::Dead)
+  MIB = BuildMI(LoadCmpBB, DL, TII->get(CMPrr))
             .addReg(DestHi, getKillRegState(Dest.isDead()))
----------------
You can probably drop the "MIB =" here. Otherwise everything looks fine though. Thanks for fixing this!


Repository:
  rL LLVM

https://reviews.llvm.org/D27315





More information about the llvm-commits mailing list