[llvm-commits] [lld] Patch for ReaderELF split into ELFAtoms.h

Alex Rosenberg alexr at leftfield.org
Fri Dec 21 16:38:57 PST 2012


On Dec 20, 2012, at 2:47 PM, Shankar Easwaran <shankare at codeaurora.org> wrote:

> Hi,
> 
> The attached patch contains changes for the below :-
> 
> a) Split ReaderELF.cpp into ELFAtoms.h so that atoms can be shared between Reader/Writer

Should there be consistency with putting the format name at the end, e.g. AtomsELF? Will other formats need their own Atoms changes like this?

Alex

> b) Changes to the DefinedAtom.h so that we can easily sort the segments and sections by descending order of permissions.
> c) Fix testcase for the behaviour to be consistent
> 
> Thanks
> 
> Shankar Easwaran
> 
> -- 
> Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation
> 
> <ReaderELF_Split_And_DefinedAtom_Changes.diff>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list