[PATCH] D118979: [AArch64] Set maximum VF with shouldMaximizeVectorBandwidth

Thu Feb 10 02:02:47 PST 2022

jaykang10 added a comment.

In D118979#3307560 <https://reviews.llvm.org/D118979#3307560>, @fhahn wrote:

>> The performance improvement from benchmarks is as below.
>
> Are the numbers SPEC scores and negative `Improvement` values regressions? If that's the case, it seems like for the benchmarks you shared the impact is negative overall?

Thanks for comments @fhahn!

The score includes a bit noise... I am sorry for that...

`505.mcf_r`, `531.deepsjeng_r`, `548.exchange2_r`, `557.xz_r` are not affected by this patch but the N1 machine showed me slightly different scores... The rest of spec2017's benchmarks have loops which affected by this patch.

I wanted to say this patch does not make overall negative impact for spec2017.

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

https://reviews.llvm.org/D118979