[PATCH] D69942: [ConstantRange][LVI] Use overflow flags from `sub` to constrain the range

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 7 05:23:34 PST 2019


lebedev.ri added a comment.

In D69942#1736906 <https://reviews.llvm.org/D69942#1736906>, @nikic wrote:

> LGTM


Thank you for the review.
Looking into `mul`/`shl`, will need to add `ConstantRange::[us]{mul,shl}_sat()` at the same time.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69942





More information about the llvm-commits mailing list