[PATCH] D37816: Experimental late jump threading pass

[Krzysztof Parzyszek via Phabricator via llvm-commits]

kparzysz added inline comments.


================
Comment at: lib/Transforms/Scalar/JumpThreading.cpp:1824
   // See the comments above FindLoopHeaders for justifications and caveats.
-  if (LoopHeaders.count(BB) || LoopHeaders.count(SuccBB)) {
+  if (LoopHeaders.count(BB) || (!IsLate && LoopHeaders.count(SuccBB))) {
     DEBUG({
----------------
bmakam wrote:
> Should this be 
> if (!IsLate && (LoopHeaders.count(BB) || LoopHeaders.count(SuccBB))) {
Jump threading over loop headers was never allowed.  You can try it and see what happens, but I'm not sure if it would be a good idea to enable it for all target.


Repository:
  rL LLVM

https://reviews.llvm.org/D37816