[PATCH] D113782: [SLP]Improve cost model for the shuffled extracts.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 1 03:01:56 PST 2021


RKSimon added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:5425
+            } while (E == getTreeEntry(Base));
+          }
+          FirstUsers.push_back(VU);
----------------
Should this be put inside the while() loop before the break? AFAICT that's the only time that cast<InsertElementInst>(Base) is valid.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113782



More information about the llvm-commits mailing list