This is an archive of the discontinued LLVM Phabricator instance.

avoid constructing an incomplete TTI in InlineCostAnnotationPrinterPass
Needs ReviewPublic

Authored by vtjnash on Feb 25 2022, 10:50 AM.

Details

Reviewers
knaumov
wsmoses
Summary

It is not an essential pass (just printing), but analysis is supposed to come from the FAM, not be constructed here (it is missing the TargetMachine, which would cause it to print the wrong costs).

Diff Detail

Event Timeline

vtjnash created this revision.Feb 25 2022, 10:50 AM
vtjnash requested review of this revision.Feb 25 2022, 10:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 25 2022, 10:50 AM
vtjnash edited the summary of this revision. (Show Details)Feb 25 2022, 10:59 AM
vtjnash added reviewers: knaumov, wsmoses.
vtjnash added a subscriber: vchuravy.
Herald added a project: Restricted Project. · View Herald TranscriptMar 8 2022, 1:57 PM

bump? this should be an easy obvious bugfix

Since you say it's possible to get the wrong cost, is it possible to create a test case that highlights this?