[PATCH] [SCEV] Improve Scalar Evolution's use of no {un, }signed wrap flags

Arnold Schwaighofer aschwaighofer at apple.com
Thu Oct 30 12:48:18 PDT 2014


LGTM.

Thanks.

================
Comment at: lib/Analysis/ScalarEvolution.cpp:6971
@@ +6970,3 @@
+    // overflow, in which case if RHS - Start is a constant, we don't need to
+    // do a max operaions since we can just figure it out statically
+    if (NoWrap && isa<SCEVConstant>(Diff)) {
----------------
s/operaions/operation/

================
Comment at: lib/Analysis/ScalarEvolution.cpp:7052
@@ +7051,3 @@
+    // overflow, in which case if RHS - Start is a constant, we don't need to
+    // do a max operaions since we can just figure it out statically
+    if (NoWrap && isa<SCEVConstant>(Diff)) {
----------------
s/operaions/operation/

http://reviews.llvm.org/D6002






More information about the llvm-commits mailing list