[PATCH] D25517: [SLPVectorizer] Improved support of partial tree vectorization.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 25 05:06:01 PST 2016


ABataev added a comment.

In https://reviews.llvm.org/D25517#602597, @hfinkel wrote:

> Have you checked the compile-time impact of this change? I wonder if you need some additional cutoff because it looks like you might be increasing the worst-case complexity of the algorithm here.


Yes, I checked it using test suite + several programs and did not find any significant changes in compile time. Actually, I thought already about some additional cutoff and I think it worth adding just in case.


https://reviews.llvm.org/D25517





More information about the llvm-commits mailing list