[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