[PATCH] D57059: [SLP] Initial support for the vectorization of the non-power-of-2 vectors.

Mon Aug 19 07:12:13 PDT 2019

ABataev added a comment.

In D57059#1635190 <https://reviews.llvm.org/D57059#1635190>, @RKSimon wrote:

> @ABataev Are you still looking at this or should we move to D66416 <https://reviews.llvm.org/D66416>? Now that X86 has enabled integer widening, we could more easily make use of this.

I just don't have enough time for this. I referenced this patch in D66416 <https://reviews.llvm.org/D66416>. This patch is not X86 specific (if integer widening is supported only for X86) + it works with the existing cost model more correctly, I believe, since our cost model is not adapted to non-power-2 vectors and must be tweaked carefully. I suggested that someone else could take this patch and split it into several smaller patches, etc. What do you think?

Repository:
  rL LLVM

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

https://reviews.llvm.org/D57059