[PATCH] D16239: Fix PR25526 by adding back limited cmpxchg pseudo-Insts

Ahmed Bougacha via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 15 18:02:42 PST 2016


ab added inline comments.

================
Comment at: lib/Target/AArch64/AArch64ExpandPseudoInsts.cpp:634
@@ +633,3 @@
+    DoneBB->addLiveIn(Reg);
+  DoneBB->splice(DoneBB->end(), &MBB, MI, MBB.end());
+
----------------
ab wrote:
> Should this also transferSuccessorsAndUpdatePHIs()?
On second thought, I guess this is late enough for transferSuccessors() to be sufficient.


http://reviews.llvm.org/D16239





More information about the llvm-commits mailing list