This is an archive of the discontinued LLVM Phabricator instance.

[ELF] Support --package-metadata
ClosedPublic

Authored by abrachet on Aug 8 2022, 2:01 PM.

Details

Summary

This was recently introduced in GNU linkers and it makes sense for
ld.lld to have the same support. This implementation omits checking if
the input string is valid json to reduce size bloat.

Diff Detail

Event Timeline

abrachet created this revision.Aug 8 2022, 2:01 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 8 2022, 2:01 PM
abrachet requested review of this revision.Aug 8 2022, 2:01 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 8 2022, 2:01 PM
MaskRay added inline comments.Aug 8 2022, 2:18 PM
lld/test/ELF/package-metadata.s
1

#

3
5

Prefer single quotes.

Please add an entry to lld/docs/ReleaseNotes.rst. Consider using a style similar to git show origin/release/15.x:lld/docs/ReleaseNotes.rst

abrachet updated this revision to Diff 450950.Aug 8 2022, 2:24 PM
abrachet marked 3 inline comments as done.

Please add an entry to lld/docs/ReleaseNotes.rst. Consider using a style similar to git show origin/release/15.x:lld/docs/ReleaseNotes.rst

Done

MaskRay accepted this revision.Aug 8 2022, 2:30 PM

Thanks!

This revision is now accepted and ready to land.Aug 8 2022, 2:30 PM
This revision was landed with ongoing or failed builds.Aug 8 2022, 2:33 PM
This revision was automatically updated to reflect the committed changes.