[PATCH] D110971: [X86][Costmodel] Load/store i8 Stride=4 VF=32 interleaving costs

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 2 03:29:21 PDT 2021


lebedev.ri added a comment.

In D110971#3038014 <https://reviews.llvm.org/D110971#3038014>, @RKSimon wrote:

> LGTM

Thank you so much for the reviews!

By now, i have to ask.
By now, all but one preexisting entries in `AVX2InterleavedLoadTbl`/`AVX2InterleavedStoreTbl` tables have been redone.
This has covered my most immediate interest. Has a saturation point been reached with these reviews,
in the sense that reviewing them become a painful boring tedious expirience? If so, i could stop here.
If not, there are a few more i could add..


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110971



More information about the llvm-commits mailing list