[PATCH] D102541: [TTI] NFC: Change getRegUsageForType to return InstructionCost.

Daniil Fukalov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 14 16:53:28 PDT 2021


dfukalov added a comment.

@sdesmalen this is needed to return from `getTypeLegalizationCost()` invalid costs, so at some point invalid legalization should be processed to a reg usage value, instead of asserting in `Optional<>::getValue()`.
What do you think, should it be processed explicitly in LV or yet in BasicTTIImplBase<>::getRegUsageForType?
In latter case, what reg usage value should it return for invalid (e.g. scalable vectors) legalization cost?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102541



More information about the llvm-commits mailing list