[PATCH] D35182: [InstCombine] remove one-use restriction for not (cmp P, A, B) --> cmp P', A, B

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 1 06:15:52 PDT 2017


spatel added a comment.

Ping.

For reference, I mentioned this patch in a related discussion about instcombine and folding of cmps:
http://lists.llvm.org/pipermail/llvm-dev/2017-July/115452.html

Based on that thread, I think we do want to proceed with this canonicalization, rather than the inverse transform suggested in:
https://bugs.llvm.org/show_bug.cgi?id=27431
...but let me know if there's a better way.


https://reviews.llvm.org/D35182





More information about the llvm-commits mailing list