[PATCH] D80992: [CostModel] Unify getArithmeticInstrCost

[Daniil Fukalov via Phabricator via llvm-commits]

dfukalov added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUTargetTransformInfo.cpp:429
   EVT OrigTy = TLI->getValueType(DL, Ty);
+  // TODO: Is this reset of these values necessary?
+  if (CostKind != TTI::TCK_RecipThroughput) {
----------------
Am I right that this reset and TCK_RecipThroughput override in the following call are needed to avoid tests failures? If so, would you please specify errors - I'll try to get a look.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80992