[PATCH] D59749: [ConstantRange] Fix isWrappedSet()

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 6 07:53:58 PDT 2019


lebedev.ri requested changes to this revision.
lebedev.ri added a comment.
This revision now requires changes to proceed.

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

> > there isn't really much left here. Could change one or two uses of isUpperWrapped() back to isWrappedSet() because it is the same in the used context...
>
> That sounds like a good cleanup to me, if you feel like it.


This differential basically fully landed.
Should be either abandoned, or rebased to contain the cleanup part.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59749





More information about the llvm-commits mailing list