[PATCH] D103144: [X86][Costmodel] Load/store v2i16 VF=2 interleaving costs

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 28 08:59:58 PDT 2021


lebedev.ri added a comment.

Thank you for taking a look!

In D103144#2787137 <https://reviews.llvm.org/D103144#2787137>, @RKSimon wrote:

> LGTM, and thanks for handling D103274 <https://reviews.llvm.org/D103274>

Could you please specify, is LGTM for this specific change;
or about the algorithm, in which case i can proceed with
adding the other missing costs in
`X86TTIImpl::getInterleavedMemoryOpCostAVX2()`
without going throught this review?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103144



More information about the llvm-commits mailing list