[PATCH] D30228: [Reassociate] Add negated value of negative constant to the Duplicates list.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 22 11:32:44 PST 2017


efriedma added a comment.

Yes, I'm fine with this approach for now.



================
Comment at: test/Transforms/Reassociate/basictest.ll:235
+  %add1 = add i64 %shl.neg, %shl.neg
+  %add2 = add i64 %add1, %b
+  ret i64 %add2
----------------
Please add CHECK lines to make sure the end result is what we expect, even if that isn't really the point of the test.


https://reviews.llvm.org/D30228





More information about the llvm-commits mailing list