[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