[PATCH] D111174: [X86][Costmodel] Improve cost modelling for not-fully-interleaved load

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 14 13:14:04 PDT 2021


lebedev.ri added a comment.

In D111174#3065041 <https://reviews.llvm.org/D111174#3065041>, @RKSimon wrote:

> In D111174#3061110 <https://reviews.llvm.org/D111174#3061110>, @lebedev.ri wrote:
>
>> While i would love/prefer to get D111546 <https://reviews.llvm.org/D111546>+D111460 <https://reviews.llvm.org/D111460> merged first, i suppose this doesn't strictly have to wait for that.
>>
>> @RKSimon does this seem like an okay intermediate step before we decide on discounting non-fully-interleaved groups?
>
> Yes, it needs rebasing after D111822 <https://reviews.llvm.org/D111822> but LGTM for committal.

Thank you for the review!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111174



More information about the llvm-commits mailing list