[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