[PATCH] D100199: [TTI] NFC: Change getCastInstrCost and getExtractWithExtendCost to return InstructionCost

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 9 09:01:31 PDT 2021


dmgreen added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/BasicTTIImpl.h:870
       // the operation will get scalarized.
-      unsigned Num = cast<FixedVectorType>(DstVTy)->getNumElements();
-      unsigned Cost = thisT()->getCastInstrCost(
+      InstructionCost Num = cast<FixedVectorType>(DstVTy)->getNumElements();
+      InstructionCost Cost = thisT()->getCastInstrCost(
----------------
Should this be a cost?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100199



More information about the llvm-commits mailing list