This is an archive of the discontinued LLVM Phabricator instance.

[LLD] [COFF] Implement support for IMAGE_REL_ARM64_ADDR32NB
ClosedPublic

Authored by mstorsjo on Oct 11 2017, 11:51 AM.

Details

Summary

Such relocations are present in object files in msvcrt.lib in Visual C++ libs for ARM64, that now is publicly available.

This is implemented in the same way as the other ADDR32NB relocations for ARM and X64.

Diff Detail

Repository
rL LLVM

Event Timeline

mstorsjo created this revision.Oct 11 2017, 11:51 AM
ruiu accepted this revision.Oct 11 2017, 12:09 PM

LGTM

COFF/Chunks.cpp
226 ↗(On Diff #118663)

Instead of adding at the end of the list, I'd add it after ADDR32, so that the list order doesn't look random.

This revision is now accepted and ready to land.Oct 11 2017, 12:09 PM
mstorsjo added inline comments.Oct 11 2017, 12:15 PM
COFF/Chunks.cpp
226 ↗(On Diff #118663)

Ok, will fix before pushing.

This revision was automatically updated to reflect the committed changes.