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

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

In D111460#3073157 <https://reviews.llvm.org/D111460#3073157>, @RKSimon wrote:

> TBH I'd much prefer we focus on fixing X86 interleaved costs, writing a script and comparing different sse levels vs cpus like I did on D103695 <https://reviews.llvm.org/D103695>, even if the llvm-mca numbers aren't great they will give us a better magnitude.
>
> Or maybe improve the fallback costs - for instance by generating costs for the BaseT getInterleavedMemoryOpCost for shuffle and scalarization patterns and returning the lowest.

As i have said these efforts are correlated, but are not identical.
I'm **not** doing this (D111460 <https://reviews.llvm.org/D111460>) to affect the interleaving costs,
i'm doing this to stop reporting dumb costs for naive gather sequence.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111460