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

[Dinar Temirbulatov via Phabricator via llvm-commits]

dtemirbulatov updated this revision to Diff 271066.
dtemirbulatov added a comment.

Addressed the comment

> Why do you need to compare flow and operation instructions count? Also, why use hardcoded `3` as a limit of vectorizable nodes?

and I removed any limitation and introduced TreeState to avoid rebuilding the tree repeatedly. Now without limitations, I could not see any regressions on compilable CPU2006 FP and Int.


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

https://reviews.llvm.org/D57779

Files:
  llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
  llvm/test/Transforms/SLPVectorizer/AArch64/gather-root.ll
  llvm/test/Transforms/SLPVectorizer/X86/load-merge.ll
  llvm/test/Transforms/SLPVectorizer/X86/powof2div.ll
  llvm/test/Transforms/SLPVectorizer/X86/slp-throttle.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57779.271066.patch
Type: text/x-patch
Size: 65693 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200616/668e4158/attachment-0001.bin>