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

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 17 08:05:03 PDT 2019


jdoerfert added a subscriber: uenoku.
jdoerfert added a comment.

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

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 :)


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

https://reviews.llvm.org/D64432





More information about the llvm-commits mailing list