This is an archive of the discontinued LLVM Phabricator instance.

[AAP] (2) Add AAP ELF definitions
AbandonedPublic

Authored by edward-jones on Aug 18 2016, 8:40 AM.

Details

Summary

This adds ELF defines and relocation for the AAP backend. This builds on D23664.

Diff Detail

Event Timeline

edward-jones retitled this revision from to [AAP] (2) Add AAP ELF definitions.
edward-jones updated this object.
edward-jones added a reviewer: rengolin.
edward-jones added subscribers: simoncook, llvm-commits.
rengolin edited edge metadata.Aug 19 2016, 2:46 AM

This looks good to me, but I'll let other people review and approve, since I was the only one on the thread. :)

edward-jones edited edge metadata.

Added ELFYaml, llvm-objdump and llvm-readobj. This also adds some tests for AAP ELF object files and relocations using the yaml representation.

Update the patch after rebasing against master

majnemer added inline comments.
include/llvm/Support/ELF.h
319–322

This looks copy+pasted, has such a request been made?
Also, you could make it 0x504141 as it would then be AAP :)

edward-jones added inline comments.Oct 3 2016, 8:09 AM
include/llvm/Support/ELF.h
319–322

The comment was copied, but a request has also been made.

Rolled forward to top of tree

joerg added a subscriber: joerg.Dec 3 2016, 3:15 PM
joerg added inline comments.
test/Object/AAP/lit.local.cfg
2 ↗(On Diff #78192)

This part is only needed until after the disassembler is added/used, there is currently nothing here that depends on the not-yet-existing target?

Incorporate Joerg's suggestions and roll forward to top of tree.

Updated to reflect latest changes to LLVM

edward-jones abandoned this revision.Jul 26 2021, 3:22 AM