[PATCH] D88505: [InstCombine] ease alignment restriction for converting masked load to normal load

Wed Sep 30 06:10:45 PDT 2020

spatel added a comment.

In D88505#2301482 <https://reviews.llvm.org/D88505#2301482>, @efriedma wrote:

> LGTM
>
> (I doubt this will come up often; most sources of "dereferenceable" also provide alignment.  But it makes sense in any case.)

I missed posting this comment before commit:
The motivation comes from increased use of masked loads that would be produced by SLP with the current proposal in D57059 <https://reviews.llvm.org/D57059>. I suspect SLP could/should do a better job of producing canonical code, but it's a mess and relies on instcombine for cleanup.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88505