This is an archive of the discontinued LLVM Phabricator instance.

[LTO] Add debug logging for module ID <-> path mapping
ClosedPublic

Authored by int3 on Mar 7 2023, 8:52 PM.

Details

Summary

It's helpful

Diff Detail

Event Timeline

int3 created this revision.Mar 7 2023, 8:52 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 7 2023, 8:52 PM
int3 requested review of this revision.Mar 7 2023, 8:52 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 7 2023, 8:52 PM
kyulee added inline comments.Mar 7 2023, 10:33 PM
llvm/lib/LTO/LTO.cpp
941

I don't think it matters too much as it's a debug code.
But if you hoist this code before readSummary above, you can get the module Id as below, and use it for both readSummary and this debug code.

uint64_t ModuleId = ThinLTO.ModuleMap.size();
MaskRay accepted this revision as: MaskRay.Mar 7 2023, 11:50 PM
This revision is now accepted and ready to land.Mar 7 2023, 11:50 PM
int3 updated this revision to Diff 503910.Mar 9 2023, 1:13 PM

address comment

This revision was landed with ongoing or failed builds.Mar 9 2023, 1:26 PM
This revision was automatically updated to reflect the committed changes.