[PATCH] D108558: [AArch64][SVE] Teach cost model masked gathers/scatters are cheap

Matt Devereau via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 25 08:07:20 PDT 2021


MattDevereau added a comment.

In D108558#2965062 <https://reviews.llvm.org/D108558#2965062>, @david-arm wrote:

> Hi @MattDevereau, given we already have tests in Analysis/CostModel/AArch64/mem-op-cost-model.ll is it necessary to sve-gather-vls.ll and sve-scatter-vls.ll tests? I was thinking you could just add those to mem-op-cost-model.ll instead.

Hi @david-arm, good point, it probably makes more sense to have them in there


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108558



More information about the llvm-commits mailing list