[PATCH] D32323: [SimplifyCFG] Fix the determination of PostBB in conditional store merging to handle the targets on the second branch being commuted

James Molloy via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 21 00:05:10 PDT 2017


jmolloy added inline comments.


================
Comment at: test/Transforms/SimplifyCFG/merge-cond-stores.ll:40
 ; This is the same as test_simple, but the branch target order has been swapped
 ; TODO: This test should succeed and end up if-converted.
 define void @test_simple_commuted(i32* %p, i32 %a, i32 %b) {
----------------
Probably can remove this TODO now?


https://reviews.llvm.org/D32323





More information about the llvm-commits mailing list