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

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 27 01:47:32 PDT 2021


lebedev.ri added a comment.

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...

> I'll resign given I neither have strong objections nor stand for it.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111460



More information about the llvm-commits mailing list