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

[Dinar Temirbulatov via Phabricator via llvm-commits]

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


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:6191-6192
     return true;
-  }
+  } else if (SLPThrottling && R.findSubTree())
+    R.saveTree();
 
----------------
ABataev wrote:
> dtemirbulatov wrote:
> > ABataev wrote:
> > > Why not try to vectorize a partial tree right here?
> > hmm, we might have an opportunity to vectorize the whole tree with smaller Chain sizes or at vectorizeStores or while doing reductions.
> Did you check that?
yes, I can add a testcase for that.


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

https://reviews.llvm.org/D57779