[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