This is an archive of the discontinued LLVM Phabricator instance.

[BOLT] Emit binary section output contents if Set
Needs ReviewPublic

Authored by yota9 on Mar 22 2023, 9:21 AM.

Details

Summary

In golang pass we're creating 2 new sections setting output contents for
them. Emit output contents if it was set during bolt work.

Vladislav Khmelevsky,
Advanced Software Technology Lab, Huawei

Diff Detail

Event Timeline

yota9 created this revision.Mar 22 2023, 9:21 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 22 2023, 9:21 AM
yota9 requested review of this revision.Mar 22 2023, 9:21 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 22 2023, 9:21 AM

Any comments? :)

Amir added a comment.Mar 31 2023, 4:23 PM

Looks benign to me. Can you please add a test for it though?

yota9 added a comment.Apr 1 2023, 12:48 AM

@Amir Sorry, but as I wrote golang pass would use this functionality, currently I don't see possibility to add the test. As we've discussed I've separated some patches since golang support is quite huge patch but not everything could be tested not all together(