[PATCH] New pass: inductive range check elimination
hfinkel at anl.gov
hfinkel at anl.gov
Tue Jan 13 17:25:18 PST 2015
In http://reviews.llvm.org/D6693#108332, @reames wrote:
> Actually, I may have accidentally misrepresented Hal's earlier comment.
> When I reread, it wasn't a clear LGTM. Please do not submit until Hal
> has okayed the current state as well.
I'm okay with this going in at this point. As has been discussed, it needs more test cases, better comments, etc., but those can be added as they're produced.
> Philip
================
Comment at: lib/Transforms/Scalar/InductiveRangeCheckElimination.cpp:652
@@ +651,3 @@
+ FailureReason = nullptr;
+
+ return true;
----------------
As a general comment, you really need some ASCII-art diagrams here illustrating what all these blocks are and how they tie together. The loop vectorizer has some of these, and while imperfect, they do help to explain what is going on.
http://reviews.llvm.org/D6693
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list