[PATCH] D100317: [TTI] NFC: Change getArithmeticInstrCost to return InstructionCost

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 14 05:32:01 PDT 2021


dmgreen accepted this revision.
dmgreen added a comment.
This revision is now accepted and ready to land.

Looks good



================
Comment at: llvm/lib/Target/X86/X86TargetTransformInfo.cpp:3422
                              CostKind)
                 .getValue() +
            getArithmeticReductionCost(Opcode, WideVecTy, IsPairwise, CostKind);
----------------
I noticed this one, which can probably be removed now too?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100317/new/

https://reviews.llvm.org/D100317



More information about the llvm-commits mailing list