This is an archive of the discontinued LLVM Phabricator instance.

[InstrProf] Don't profile merge by default in lightweight mode
ClosedPublic

Authored by ellis on Dec 17 2021, 3:48 PM.

Details

Summary

Profile merging is not supported when using debug info profile
correlation because the data section won't be in the binary at runtime.
Change the default profile name in this mode to default_%p.proflite so
we don't use profile merging.

Diff Detail

Event Timeline

ellis created this revision.Dec 17 2021, 3:48 PM
ellis published this revision for review.Dec 17 2021, 3:50 PM
ellis added reviewers: kyulee, MaskRay, alanphipps.
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptDec 17 2021, 3:51 PM
Herald added subscribers: Restricted Project, cfe-commits. · View Herald Transcript

For clarification, the lightweight pgo does not support merging raw data from memory (at runtime) due to missing structural data, but it does support merging raw files offline.
Would you update the comment or error message?

kyulee accepted this revision.Dec 18 2021, 7:14 PM
This revision is now accepted and ready to land.Dec 18 2021, 7:14 PM
ellis updated this revision to Diff 395481.Dec 20 2021, 9:50 AM

Expand warning message

This revision was landed with ongoing or failed builds.Dec 20 2021, 9:51 AM
This revision was automatically updated to reflect the committed changes.