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

[Krzysztof Parzyszek via Phabricator via llvm-commits]

kparzysz added a comment.

In D101533#2726569 <https://reviews.llvm.org/D101533#2726569>, @sdesmalen wrote:

> For this patch it's probably best to just follow precedent as to what was used before, i.e. cases that used `std::pair<unsigned, MVT>` should be changed to `std::pair<InstructionCost, MVT>`

I disagree with this part.  If there is a name for this type then we should use it, instead of repeating the definition.  Either `TargetLoweringBase::LegalizeCost` or `TargetLowering::LegalizeCost`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101533