[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