[PATCH] D110739: [CostModel] Update default cost model for sadd/ssub overflow to match TargetLowering
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 29 15:09:07 PDT 2021
craig.topper added inline comments.
================
Comment at: llvm/include/llvm/CodeGen/BasicTTIImpl.h:1797
InstructionCost Cost = 0;
Cost += thisT()->getArithmeticInstrCost(Opcode, SumTy, CostKind);
Cost += 2 * thisT()->getCmpSelInstrCost(
----------------
RKSimon wrote:
> Worth trying to specify the Args as well via the IntrinsicCostAttributes arg?
You mean to get the getArithmeticInstrCost calls? I don't see any arg operands for getCmpSelInstrCost.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110739/new/
https://reviews.llvm.org/D110739
More information about the llvm-commits
mailing list