[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