[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