[PATCH] D112598: [WIP][RISCV][CostModel] Add cost model for shuffle

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp:178
+  if (Kind == TTI::SK_Broadcast) {
+    return TLI->getTypeLegalizationCost(DL, Tp).first;
+  }
----------------
Should this consider LMUL? The ALU likely needs more cycles for LMUL 2 than LMUL 1.


================
Comment at: llvm/test/Analysis/CostModel/RISCV/shuffle-broadcast.ll:1
+; NOTE: Assertions have been autogenerated by utils/update_analyze_test_checks.py
+; RUN: opt < %s -mtriple=riscv64--linux-gnu -mattr=+experimental-v,+f -cost-model -analyze | FileCheck %s
----------------
Doesn’t this need -riscv-v-vector-bits-min?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112598