[PATCH] D111942: [X86][Costmodel] Load/store i32 Stride=3 VF=32 interleaving costs

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 17 05:25:16 PDT 2021


lebedev.ri added a comment.

In D111942#3068918 <https://reviews.llvm.org/D111942#3068918>, @RKSimon wrote:

> @lebedev.ri I've created D111960 <https://reviews.llvm.org/D111960> to see if we can reduce these costs a little - given how manual their testing is, it'd be great to ensure the codegen is reasonable before you do any more work on them.

Sure, but for these strides these patches only finish the few missing pieces,
while after codegen improves we may optionally have to go back and update the costs
and a few more items to update won't really affect things.
The next big batch (stride=5/7/8) can indeed wait a bit for better codegen.

After these, hopefully D111460 <https://reviews.llvm.org/D111460> lands, and then i think i'd like to decide on discounting not-fully-interleaved loads before finishing with the missing strides.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111942



More information about the llvm-commits mailing list