[PATCH] D70789: [Attributor] Remove dereferenceable_or_null when nonull is present

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 28 10:39:29 PST 2019


lebedev.ri added a comment.

In D70789#1763283 <https://reviews.llvm.org/D70789#1763283>, @jdoerfert wrote:

> LGTM.
>
> In D70789#1762474 <https://reviews.llvm.org/D70789#1762474>, @uenoku wrote:
>
> > In D70789#1762114 <https://reviews.llvm.org/D70789#1762114>, @jdoerfert wrote:
> >
> > > This was a problem for a while now:
> > >  Can we have a test case to make sure we do not lose information, e.g. `deref_or_null(100) nonnull deref(4)` should not remove the `deref_or_null(100)`.
> >
>


More generally, we do already merge `deref_or_null(x) deref(y)` -> `deref(max(x, y))` ?


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

https://reviews.llvm.org/D70789





More information about the llvm-commits mailing list