[PATCH] D111460: [X86][LoopVectorize] "Fix" `X86TTIImpl::getAddressComputationCost()`

Wed Oct 27 01:51:40 PDT 2021

pengfei added a comment.

In D111460#3089621 <https://reviews.llvm.org/D111460#3089621>, @lebedev.ri wrote:

> In D111460#3089603 <https://reviews.llvm.org/D111460#3089603>, @pengfei wrote:
>
>>> @pengfei like with D111546 <https://reviews.llvm.org/D111546>, are intel benchmarks impacted by this? in which way?
>>
>> I checked cpu2017 on SKX, no obvious improvment/regression found.
>
> Thank you for checking! This is mainly for the CPU's with no/poor gather support,
> IOW as it currently stands, those without AVX512. But i think `SKX` in LLVM
> is `Skylake-AVX512`? So those results aren't really representable...

Yes, but I don't have the target without fast gather support.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111460