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

[Shankar Easwaran]

Hi Alex,

I get your point on changing the filename, may be I should do that.

Thanks

Shankar Easwaran

On 12/21/2012 6:38 PM, Alex Rosenberg wrote:
> 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


-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation