[PATCH] D20379: Codegen: Fix broken assumption in Tail Merge.

Kyle Butt via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 8 16:59:52 PDT 2016


iteratee added a comment.

I updated the test cases as requested.


================
Comment at: lib/CodeGen/BranchFolding.cpp:969
@@ -966,1 +968,3 @@
     MachineBasicBlock *PredBB = &*std::prev(I);
+    if (PredBB && !PredBB->isSuccessor(IBB))
+      PredBB = nullptr;
----------------
That to me implies that it will be null if there is no such predecessor.
I need to know if you want me to remove it from this patch.


http://reviews.llvm.org/D20379





More information about the llvm-commits mailing list