[PATCH] D113770: [SLP]Improve cost of the gather nodes.

[Vasileios Porpodas via Phabricator via llvm-commits]

vporpo added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:5570
   auto *VecTy = FixedVectorType::get(ScalarTy, VL.size());
+  bool NeedToShuffle = false;
   // Find the cost of inserting/extracting values from the vector.
----------------
Could you a comment here mentioning that gathering from constants does not need shuffles? Or perhaps make it more explicit by renaming this variable to GatherFromConstants = true and resetting it if any of the VL elements is not a constant?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113770