[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