This is an archive of the discontinued LLVM Phabricator instance.

Object: Add ELF types to ELFType.
ClosedPublic

Authored by ruiu on Mar 12 2016, 10:48 PM.

Details

Reviewers
rafael
Summary

These types are defined in ELFFile, so in order to use them, you have
to write ELFFile<ELFT>::SomeType. But there seems to be no reason to have
ELFFile have these types. This patch allows you to write ELFT::SomeType
instead.

This simplifies libObject users.
This is an example: http://reviews.llvm.org/D18129

Diff Detail

Event Timeline

ruiu updated this revision to Diff 50544.Mar 12 2016, 10:48 PM
ruiu retitled this revision from to Object: Add ELF types to ELFType..
ruiu updated this object.
ruiu added a reviewer: rafael.
ruiu added a subscriber: llvm-commits.
rafael accepted this revision.Mar 14 2016, 3:50 PM
rafael edited edge metadata.

I really like this!

Please just make sure it builds with gcc given the "using packed =".

This revision is now accepted and ready to land.Mar 14 2016, 3:50 PM
ruiu added a subscriber: ruiu.Mar 14 2016, 4:01 PM

It builds fine with gcc. Submitting.

ruiu closed this revision.Mar 14 2016, 4:03 PM