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

[Craig Topper via Phabricator via llvm-commits]

craig.topper 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) {
----------------
jmolloy wrote:
> Probably can remove this TODO now?
Yeah. That was my intention when I put in the TODO but forgot. Thank!


https://reviews.llvm.org/D32323