This is an archive of the discontinued LLVM Phabricator instance.

llvm - more AArch64 relocations added
AbandonedPublic

Authored by pawelo on Jul 13 2014, 10:39 AM.

Details

Reviewers
t.p.northover
Summary

I've found that ELF.h file lacks few AArch64 relocations listed here (and other places I can easily google): http://code.metager.de/source/xref/gnu/grub/include/grub/elf.h

Diff Detail

Event Timeline

pawelo updated this revision to Diff 11354.Jul 13 2014, 10:39 AM
pawelo retitled this revision from to llvm - more AArch64 relocations added.
pawelo updated this object.
pawelo edited the test plan for this revision. (Show Details)
pawelo set the repository for this revision to rL LLVM.
pawelo added a subscriber: Unknown Object (MLST).
t.p.northover accepted this revision.Jul 13 2014, 11:49 AM
t.p.northover added a reviewer: t.p.northover.
t.p.northover added a subscriber: t.p.northover.

Hi Paul,

Looks fine to me (and the most important place is http://infocenter.arm.com/help/topic/com.arm.doc.ihi0056b/IHI0056B_aaelf64.pdf, incidentally).

Cheers.

Tim.

This revision is now accepted and ready to land.Jul 13 2014, 11:49 AM
pawelo abandoned this revision.Aug 12 2014, 2:28 AM

Tim Northover commited later patch by Daniel Stewart which does the same thing.