[PATCH] D115757: [SLP]Generalize cost model.

Wed Nov 30 03:42:05 PST 2022

ABataev added a comment.

In D115757#3959221 <https://reviews.llvm.org/D115757#3959221>, @LuoYuanke wrote:

> In D115757#3958922 <https://reviews.llvm.org/D115757#3958922>, @ABataev wrote:
>
>> In D115757#3958787 <https://reviews.llvm.org/D115757#3958787>, @LuoYuanke wrote:
>>
>>> It seems there is regression for 538 in your data, and we observed the regression for cpu2017/638 too. Could you fix the regression for them?
>>>
>>> `538.imagick_r           -1.42`
>>
>> Not as an immediate fix, hope future changes will fix it.
>
> Do you have any plan to fix it?

A plan of improving SLP vectorizer, not this particular regression. It will be fixed eventually

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115757