[PATCH] D16809: [JumpThreading] See through Cast Instructions

Geoff Berry via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 5 11:01:26 PST 2016


gberry added inline comments.

================
Comment at: test/Transforms/JumpThreading/basic.ll:493
@@ +492,3 @@
+
+Merge:
+	%B = phi i32 [1, %Entry], [%v1, %F1]
----------------
It seems like this change to JumpThreading would not be needed if the Merge block code had been simplified to the following:

```
Merge:
  %B = phi i32 [1, %Entry], [%v1, %F1]
  %M = icmp eq i32 %B, 0 
  br i1 %M, label %T2, label %F2
```

Are we missing this simplification, or is there something more complex going on in the code this was extracted from that prevents it from happening?


Repository:
  rL LLVM

http://reviews.llvm.org/D16809





More information about the llvm-commits mailing list