[PATCH] D22779: Clone block with icmp+branch if it likely results in further jump threading

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 30 18:55:11 PDT 2016


reames added a subscriber: reames.

================
Comment at: lib/Transforms/Scalar/JumpThreading.cpp:885
@@ -884,1 +884,3 @@
 
+  // If this is an otherwise-unthreadable conditional branch on a compare,
+  // duplicate this into its predecessor if that will result in the value of the
----------------
This comments needs clarified.  I was going to do a quick drive by style review, but after reading this three times, I'm not clear what you're trying to do.  That's a pretty good sign the comment needs adjusted. :)

I think you've mainly got a grammar problem; this reads as one long run on sentence which is hard to parse.  


https://reviews.llvm.org/D22779





More information about the llvm-commits mailing list