[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