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

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 16 12:36:28 PDT 2019


spatel added a comment.

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

> Could we check here if the base pointer has `dereferenceable` annotation and use that as a condition for this transformation? (It's more complicated to be completely lossless but this seems to be an easy to test starting point).


That seems like a reasonable way to overcome the potential loss, so I'll update this patch with that change.
I'm expecting that will significantly reduce the opportunities for the transform though...until Attributor is working at full strength.


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

https://reviews.llvm.org/D64432





More information about the llvm-commits mailing list