This is an archive of the discontinued LLVM Phabricator instance.

[obj2yaml] [COFF] Write RVA instead of VA for sections, fix roundtripping executables
ClosedPublic

Authored by mstorsjo on Nov 27 2018, 11:54 AM.

Details

Summary

This is an alternative to D54946, both fixing roundtripping executables. This patch changes the serialization format to be RVA, matching what's stored in files.

Diff Detail

Repository
rL LLVM

Event Timeline

mstorsjo created this revision.Nov 27 2018, 11:54 AM
zturner accepted this revision.Nov 29 2018, 9:21 AM

Ahh yes, this is the other patch you were referring to. I like this one too.

This revision is now accepted and ready to land.Nov 29 2018, 9:21 AM
This revision was automatically updated to reflect the committed changes.