[PATCH] D106795: [Analysis] Fix getOrderedReductionCost to call target's getArithmeticInstrCost implementation
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 26 08:11:44 PDT 2021
RKSimon added inline comments.
================
Comment at: llvm/include/llvm/CodeGen/BasicTTIImpl.h:2093
InstructionCost ExtractCost =
getScalarizationOverhead(VTy, /*Insert=*/false, /*Extract=*/true);
+ InstructionCost ArithCost = thisT()->getArithmeticInstrCost(
----------------
Just to be clear - this getScalarizationOverhead call isn't replaced by a target version, its just a base class wrapper that calls the target's DemandedElts variant of getScalarizationOverhead
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106795/new/
https://reviews.llvm.org/D106795
More information about the llvm-commits
mailing list