[PATCH] D96818: [SLP]No need to mark scatter load pointer as scalar as it gets vectorized.
Anton Afanasyev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 17 07:40:03 PST 2021
anton-afanasyev accepted this revision.
anton-afanasyev added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:2578
if (UseScalar != U ||
+ UseEntry->State == TreeEntry::ScatterVectorize ||
!InTreeUserNeedToExtract(Scalar, UserInst, TLI)) {
----------------
We could make the same check inside `InTreeUserNeedToExtract()`, but here it looks more simple.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96818/new/
https://reviews.llvm.org/D96818
More information about the llvm-commits
mailing list