Page MenuHomePhabricator

[ELF2] Initial (very basic) support for AArch64
ClosedPublic

Authored by davide on Sep 25 2015, 10:44 AM.

Details

Summary

We can link now!

root@cavium:/zoo/davide # file ./helloarch
./helloarch: ELF 64-bit LSB executable, ARM aarch64, version 1 (FreeBSD), statically linked, not stripped
root@cavium:/zoo/davide # ./helloarch
root@cavium:/zoo/davide # echo $?
42

Diff Detail

Repository
rL LLVM

Event Timeline

davide updated this revision to Diff 35737.Sep 25 2015, 10:44 AM
davide retitled this revision from to [ELF2] Initial (very basic) support for AArch64.
davide updated this object.
davide added a reviewer: rafael.
davide set the repository for this revision to rL LLVM.
davide added a subscriber: llvm-commits.
ruiu accepted this revision.Sep 25 2015, 10:49 AM
ruiu added a reviewer: ruiu.
ruiu added a subscriber: ruiu.

Cool! LGTM but wait for Rafael.

This revision is now accepted and ready to land.Sep 25 2015, 10:49 AM
emaste added a subscriber: emaste.Sep 25 2015, 10:54 AM

Excellent! Aside, the reference FreeBSD/arm64 platform where @davide ran the result is Cavium's ThunderX.

ELF/SymbolTable.cpp
48–49 ↗(On Diff #35737)

My usual comment, we ought to keep lists like this sorted in alphanumeric order. Perhaps just put EM_AARCH64 first in this commit and re-sort the rest later?

rafael accepted this revision.Sep 25 2015, 1:59 PM
rafael edited edge metadata.

LGTM. Please sort the list in a followup commit.

This revision was automatically updated to reflect the committed changes.