[PATCH] D54411: [Codegen] Merge tail blocks with no successors after block placement

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 3 02:19:16 PDT 2019


RKSimon added inline comments.


================
Comment at: test/CodeGen/X86/tail-opts.ll:439
+  %1 = icmp eq i32 undef, 0
+  br i1 %1, label %return, label %bb12
 
----------------
dmgreen wrote:
> Can you explain this change? The comment makes it look like it should be the same test as above.
> 
> But it somehow has different fall-through branches with the switches?
icmp with undef will now constant fold in SelectionDAG - you're probably being affected by that


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54411/new/

https://reviews.llvm.org/D54411





More information about the llvm-commits mailing list