[PATCH] D133007: [RISCV] Add cost model for vector insert/extract element.

Jianjian Guan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 1 19:15:15 PDT 2022


jacquesguan added a comment.

In D133007#3764086 <https://reviews.llvm.org/D133007#3764086>, @reames wrote:

> Adding the insertelement handling caused a lot more test diffs.  For ease of review, would you mind restricting this patch to extractelement and then doing insertelement in a follow up patch?

The reason why other test cases affected is that some caculation of other instructions' cost depends on the cost of insertelement and extractelement. It my mistake to forget upload them before. So splitting this revision would be not help for reviewing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133007



More information about the llvm-commits mailing list