[PATCH] D78216: [TTI] Add DemandedElts to getScalarizationOverhead

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 27 14:34:13 PDT 2020


craig.topper added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:3881
+      DemandedElts.setBit(i);
+  int Cost = TTI->getScalarizationOverhead(Ty, DemandedElts, true, false);
   if (!ShuffledIndices.empty())
----------------
Can we annotate the bools with /**/ comments in all the call sites to make it readable?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78216





More information about the llvm-commits mailing list