Page MenuHomePhabricator

NFC: Change getTypeBasedIntrinsicCost to return InstructionCost
ClosedPublic

Authored by sdesmalen on Feb 25 2021, 6:33 AM.

Details

Summary

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

See this patch for the introduction of the type: https://reviews.llvm.org/D91174
See this thread for context: http://lists.llvm.org/pipermail/llvm-dev/2020-November/146408.html

Depends on D97466

Diff Detail

Event Timeline

sdesmalen created this revision.Feb 25 2021, 6:33 AM
sdesmalen requested review of this revision.Feb 25 2021, 6:33 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 25 2021, 6:33 AM
dmgreen accepted this revision.Feb 28 2021, 10:41 AM
dmgreen added a subscriber: dmgreen.

LGTM, providing the *getValue() are removed again once the interface is updated.

This revision is now accepted and ready to land.Feb 28 2021, 10:41 AM

LGTM, providing the *getValue() are removed again once the interface is updated.

Thanks! Yes, the *getValue()'s are indeed removed in the follow-up patch D97469.