[PATCH] D20058: [SCEV] No-wrap flags are not propagated when folding "{S, +, X}+T ==> {S+T, +, X}"

[Sanjoy Das via llvm-commits]

sanjoy accepted this revision.
sanjoy added a comment.
This revision is now accepted and ready to land.

lgtm with nits

Let me know if you need me to check this in for you.


================
Comment at: test/Analysis/ScalarEvolution/nsw.ll:181
@@ +180,3 @@
+
+; This test checks if no-wrap flags are propagated when folding {S,+,X}+T ==> {S+T,+,X} (D20058)
+; CHECK: %idxprom
----------------
Nit: you don't need to specify `D20058` here.  Also use a `CHECK-LABEL: <function name>` like in the previous tests.

================
Comment at: test/Analysis/ScalarEvolution/nsw.ll:201
@@ +200,2 @@
+  ret void
+}
----------------
Add newline at end of file?


Repository:
  rL LLVM

http://reviews.llvm.org/D20058