Page MenuHomePhabricator

[TTI] NFC: Change getCastInstrCost and getExtractWithExtendCost to return InstructionCost

Authored by sdesmalen on Fri, Apr 9, 7:58 AM.



This patch migrates the TTI cost interfaces to return an InstructionCost.

See this patch for the introduction of the type:
See this thread for context:

Diff Detail

Event Timeline

sdesmalen created this revision.Fri, Apr 9, 7:58 AM
sdesmalen requested review of this revision.Fri, Apr 9, 7:58 AM
Herald added a project: Restricted Project. · View Herald TranscriptFri, Apr 9, 7:58 AM
dmgreen added inline comments.Fri, Apr 9, 9:01 AM

Should this be a cost?

sdesmalen added inline comments.Fri, Apr 9, 9:31 AM

It should not, good spot!

sdesmalen updated this revision to Diff 336496.Fri, Apr 9, 9:32 AM

Revert InstructionCost change for Num variable.

dmgreen accepted this revision.Sat, Apr 10, 6:49 AM

Thanks. LGTM

This revision is now accepted and ready to land.Sat, Apr 10, 6:49 AM