[PATCH] D105432: [Analysis] Add simple cost model for strict (in-order) reductions
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 26 07:42:04 PDT 2021
RKSimon added inline comments.
================
Comment at: llvm/include/llvm/CodeGen/BasicTTIImpl.h:2095
+ InstructionCost ArithCost =
+ getArithmeticInstrCost(Opcode, VTy->getElementType(), CostKind);
+ ArithCost *= VTy->getNumElements();
----------------
@david-arm I think this needs to be thisT()->getArithmeticInstrCost to use the correct TTI implementation?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105432/new/
https://reviews.llvm.org/D105432
More information about the llvm-commits
mailing list