[PATCH] D64432: [InstCombine] try to narrow a truncated load

Wed Jul 17 08:46:55 PDT 2019

spatel added a comment.

In D64432#1589539 <https://reviews.llvm.org/D64432#1589539>, @jdoerfert wrote:

> I think you are missing the negative test without `dereferenceable`.

Yes - updated.

> In D64432#1588103 <https://reviews.llvm.org/D64432#1588103>, @spatel wrote:
> 
>> I'm expecting that will significantly reduce the opportunities for the transform though...until Attributor is working at full strength.
> 
> 
> Agreed. Though, we already have `dereferenceabl` arguments. @uenoku Is now working on `dereferenceable` deduction for the Attributor :)

Great!

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

https://reviews.llvm.org/D64432