[PATCH] D105220: [ScalarizeMaskedMemIntrin] Use the element type to calculate alignment for gather/scatter when alignment operand is 0.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

In D105220#2850946 <https://reviews.llvm.org/D105220#2850946>, @lebedev.ri wrote:

> I think this is correct.
> I guess for that you need to pass `0` into `alignment` argument of these intrinsics.

Yeah. And to have an observable change you need a target with a cost model that returns a different answer if you give it element alignment versus vector alignment. But since gather works on elements, I would expect that any cost model that cares about alignment is checking whether it is element aligned or not.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105220