[PATCH] D101533: [TTI] NFC: Change getTypeLegalizationCost to return InstructionCost.

Daniil Fukalov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 29 11:35:15 PDT 2021


dfukalov added a comment.

So what do you think, should I use `TargetLoweringBase::LegalizeCost` as type everywhere (and it doesn't seem clearer than `auto`) or just remove the new type and use `std::pair<InstructionCost, MVT>` everywhere?

I just saw a number `auto LT = TLI->getTypeLegalizationCost()` in AArch64 backend and decided it is ok.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101533



More information about the llvm-commits mailing list