[PATCH] Make ScalarEvolution less aggressive with respect to no-wrap flags.

Andrew Trick atrick at apple.com
Wed Jan 21 09:00:21 PST 2015


LGTM with one suggestion for comment.
Thanks for the test cases.


================
Comment at: lib/Analysis/ScalarEvolution.cpp:3464-3469
@@ -3463,8 +3464,2 @@
                 }
-              } else if (const SubOperator *OBO =
-                           dyn_cast<SubOperator>(BEValueV)) {
-                if (OBO->hasNoUnsignedWrap())
-                  Flags = setFlags(Flags, SCEV::FlagNUW);
-                if (OBO->hasNoSignedWrap())
-                  Flags = setFlags(Flags, SCEV::FlagNSW);
               }
----------------
Add a short comment that subtract does not preserve nsw/nuw Flags.

http://reviews.llvm.org/D7081

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list