[PATCH] D152001: [RISCV][SLP] Inflate insert/extract costs on very small vectors

Thu Jul 20 10:39:49 PDT 2023

reames added a comment.

In D152001#4511274 <https://reviews.llvm.org/D152001#4511274>, @ABataev wrote:

> In D152001#4511205 <https://reviews.llvm.org/D152001#4511205>, @reames wrote:
>
>> @ABataev ping?
>
> Check D154738 <https://reviews.llvm.org/D154738>

I went and dug through this patch.  I really don't think your patch and mine are solving the same problem at all.  Yours appear to be focused on whether we should have vectorized an operation based on it's legality.  Mine is focused on biasing vectorization away from *legal* but *unprofitable* partial vectorization scenarios.  There's definitely an interaction here, but the approach is quite different.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152001