[PATCH] D57779: [SLP] Add support for throttling.

Dinar Temirbulatov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 20 17:11:21 PDT 2019


dtemirbulatov marked an inline comment as done.
dtemirbulatov added inline comments.


================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:3521
+  // Reduce the path from branch L to find a profitable cut.
+  for (unsigned I = 0; I < Path.size(); I++) {
+    SmallVector<unsigned, 2> SubTree(Vecs);
----------------
vporpo wrote:
> This is calculating every possible cut in the path to the leaf. However, some paths will probably share the nodes close to the root, so we are recomputing the same thing multiple times.
oh, I missed that in this version completely, Thanks.


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

https://reviews.llvm.org/D57779





More information about the llvm-commits mailing list