[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