[llvm-commits] Proof of concept patch for unifying the .s/ELF emission of .ARM.attributes

[Jason Kim]

On Wed, Oct 13, 2010 at 10:23 AM, Jim Grosbach <grosbach at apple.com> wrote:
>
> On Oct 12, 2010, at 10:53 AM, Jason Kim wrote:
>
>> Thanks Jim!
>>
>> So I guess I'll do the following.
>> 1. Fix up elf-dump via Rafael's simplification.
>> 2. Tweak/Resend this patch in sequence to this list.,
>> 3. Commit?
>>
>> Does this still sound good to you guys?
>
>
> Works for me. Looking forward to seeing the next bits!

Rafael: elf-dump has been simplified as per your suggestion - --hex
wil not be default until remaining tests are refactored.

---
Please find enclosed two patches
arm-mc-elf-s06-elfdump.patch2 modifies elf-dump script.
It adds a --hex mode for printing all numbers to hex format.
AFAIK, elf-dump output without --hex has remained the same (existing
MC tests all pass)

The second patch arm-mc-elf-s06.patch adds enough code to directly
test ARM/MC/ELF/.o emission.
The .ARM.attributes section is now identical to GNU/as for cases where
.cpu == "generic"
TODO:
* correct the initial ELF headers for ARM

Unless there's strong objection, I'd like to land these soon.

Thanks!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: arm-mc-elf-s06-elfdump.patch2
Type: application/octet-stream
Size: 7937 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20101015/284b7b96/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: arm-mc-elf-s06.patch3
Type: application/octet-stream
Size: 12766 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20101015/284b7b96/attachment-0001.obj>