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

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added a comment.

In D57059#2578657 <https://reviews.llvm.org/D57059#2578657>, @anton-afanasyev wrote:

> Btw, how could it be explained `NumVectorInstructions` stat reducing after this patch?
>
> In D57059#2553914 <https://reviews.llvm.org/D57059#2553914>, @ABataev wrote:
>
>> Extra numbers:
>>
>> **AVX512, O3 <https://reviews.llvm.org/owners/package/3/>+LTO, -march=native**
>> Metric: SLP.NumVectorInstructions
>>
>>   ...
>>         test-suite :: External/SPEC/CFP2017speed/638.imagick_s/638.imagick_s.test  3996.00  3563.00 -10.8%
>>          test-suite :: External/SPEC/CFP2017rate/538.imagick_r/538.imagick_r.test  3996.00  3563.00 -10.8%
>>                 test-suite :: External/SPEC/CINT2017rate/502.gcc_r/502.gcc_r.test   862.00   767.00 -11.0%
>>                test-suite :: External/SPEC/CINT2017speed/602.gcc_s/602.gcc_s.test   862.00   767.00 -11.0%
>>         test-suite :: External/SPEC/CINT2017rate/520.omnetpp_r/520.omnetpp_r.test   524.00   463.00 -11.6%
>>   ...

Actually, it is not reducing. This is how test-suite python script works. So, here lhs - number of instructions after this patch, rhs - before. And the less relative number, the more vector instructions we actually generate.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D57059