This is an archive of the discontinued LLVM Phabricator instance.

[TTI] Fix ScalarizationCost initialization.
ClosedPublic

Authored by dfukalov on Apr 22 2021, 12:19 PM.

Details

Summary

In cases when ScalarizationCostPassed has no value, UINT_MAX is actually used
for cost estimation in return ScalarCalls * ScalarCost + ScalarizationCost.

Diff Detail

Event Timeline

dfukalov requested review of this revision.Apr 22 2021, 12:19 PM
dfukalov created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptApr 22 2021, 12:19 PM
sdesmalen accepted this revision.Apr 23 2021, 7:41 AM

This indeed seems to fix a bug in the cost-model, LGTM.

This revision is now accepted and ready to land.Apr 23 2021, 7:41 AM
This revision was landed with ongoing or failed builds.Apr 23 2021, 8:00 AM
This revision was automatically updated to reflect the committed changes.