[PATCH] D39835: [GVN PRE] Clear nsw/nuw for original values in LoadPRE

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 9 12:34:56 PST 2017


efriedma added inline comments.


================
Comment at: test/Transforms/GVN/PRE/pre-jt-load.ll:26
+merge:
+  %foo = load i32, i32* @G, align 4
+  %cmp = icmp sgt i32 %foo, 0
----------------
Transforming this load into a phi in GVN is fine without stripping nsw.  That should be more obvious if you replace "%v" in one of the additions with a different value.


https://reviews.llvm.org/D39835





More information about the llvm-commits mailing list