[PATCH] D99355: Implementation of intrinsic and SDNode definitions for VP load, store, gather, scatter.

Thu May 27 10:02:32 PDT 2021

hussainjk added a comment.

In D99355#2784442 <https://reviews.llvm.org/D99355#2784442>, @simoll wrote:

> In D99355#2781282 <https://reviews.llvm.org/D99355#2781282>, @frasercrmck wrote:
>
>> I had a thought about alignment. Do we have a plan on how to deal with unaligned accesses?
>
> Not yet
>
>> The other non-VP masked intrinsics are scalarized by the `ScalarizeMaskedMemIntrin` pass and regular loads and stores are handled by -- I believe -- `LegalizeDAG`.
>
> Maybe vp can piggyback on `ScalarizeMaskedMemIntrin` for unaligned access expansion.

It already will piggyback on `ScalarizeMaskedMemIntrin` for scalarization, so that should be straightforward to put into the followup VP-mem-expansion patch.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99355