This is an archive of the discontinued LLVM Phabricator instance.

[TextAPI][elfabi] Fix YAML support for weak symbols
ClosedPublic

Authored by amontanez on Dec 21 2018, 12:19 PM.

Details

Summary

Weak symbols are supposed to be supported in the ELF TextAPI implementation, but the YAML handler didn't read or write the Weak member of ELFSymbol. This change adds the YAML mapping and updates tests to ensure correct behavior.

Diff Detail

Repository
rL LLVM

Event Timeline

amontanez created this revision.Dec 21 2018, 12:19 PM
phosek accepted this revision.Dec 21 2018, 12:31 PM

LGTM

This revision is now accepted and ready to land.Dec 21 2018, 12:31 PM
This revision was automatically updated to reflect the committed changes.