[PATCH] D90445: [SLP] Make SLPVectorizer to use `llvm.masked.gather` intrinsic

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 9 08:49:02 PST 2020


ABataev added a comment.

In D90445#2382105 <https://reviews.llvm.org/D90445#2382105>, @anton-afanasyev wrote:

> In D90445#2380322 <https://reviews.llvm.org/D90445#2380322>, @ABataev wrote:
>
>> In D90445#2380023 <https://reviews.llvm.org/D90445#2380023>, @anton-afanasyev wrote:
>>
>>> Added second tree entry for addresses
>>
>> Yep, this looks much better now. What about adding new vectorization kind for scatter-gather?
>
> What do you mean by "vectorization kind"? Vectorization is differentiated by bundle instructions opcode.

I meant, to add a new vectorization kind for this node, something like `Scatter` or `ScatterVectorized`. We discussed before, that this new boolean field is going to e unused in many cases and better to introduce a new vectorization id (extend `EntryState` enum)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90445



More information about the llvm-commits mailing list