[PATCH] InstCombine: propagate deref via new addDereferenceableAttr

Philip Reames listmail at philipreames.com
Mon Feb 9 15:32:03 PST 2015


In http://reviews.llvm.org/D7510#120890, @artagnon wrote:

> In http://reviews.llvm.org/D7510#120855, @reames wrote:
>
> > p.s. isDerefPtr is on Value, the cast is entirely unnecessary.
>
>
> Ah, but `->getDereferenceableBytes()` is only defined for an `Argument`; I'm not sure why it is that way though.


Ok, I see why you need the cast.  Another idea for later cleanup.  :)


http://reviews.llvm.org/D7510

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list