[PATCH] D135174: [SLP]Redesign vectorization of the gather nodes.
Alexey Bataev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 4 08:00:31 PDT 2022
ABataev added inline comments.
================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:8273
+ if (I != Pos)
+ Gathered[I] = PoisonValue::get(Gathered[I]->getType());
+ });
----------------
RKSimon wrote:
> Please can you explain what this poisonvalue is for and why we don't need freeze for it.
It is to build the the shuffle with the poison only. On line 8271 the corresponding shuffle position is set to the non-poisoned element from the buildvector. The freeze is not required because we already checked that the scalar in Pos position is non-poisoned.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135174/new/
https://reviews.llvm.org/D135174
More information about the llvm-commits
mailing list