[PATCH] D60632: [ConstantRange] Disallow NUW | NSW in makeGuaranteedNoWrapRegion()

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 13 11:00:09 PDT 2019


sanjoy added a comment.

In D60632#1465299 <https://reviews.llvm.org/D60632#1465299>, @lebedev.ri wrote:

> That was added in D13612 <https://reviews.llvm.org/D13612> `This will be used in a future change to ScalarEvolution.`,
>  but indeed, the `(OBO::NoSignedWrap | OBO::NoUnsignedWrap)` combination is not used anywhere.
>  I'm wondering if @sanjoy has any comments?


I don't remember if I had any specific plans for `(OBO::NoSignedWrap | OBO::NoUnsignedWrap)` but please remove it if it is unused today.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60632





More information about the llvm-commits mailing list