[PATCH] D105209: [WIP][SCEVExpander] Discount cost of umin(x,1) expressions

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 30 10:35:43 PDT 2021


reames planned changes to this revision.
reames added a comment.

For future self:

- and reduction trick does not hold when A and B are both non-zero, but with different bits set.
- the original umin(a,1) trick still holds, it's the extension to multiple operands which was wrong




CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105209/new/

https://reviews.llvm.org/D105209



More information about the llvm-commits mailing list