[PATCH] D110739: [CostModel] Update default cost model for sadd/ssub overflow to match TargetLowering

[Craig Topper via Phabricator via llvm-commits]

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