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

[Jason Kim]

On Tue, Oct 12, 2010 at 10:48 AM, Jim Grosbach <grosbach at apple.com> wrote:
>
> On Oct 12, 2010, at 10:40 AM, Jason Kim wrote:
>>
>> I think I'll separate that out into 1+ new patches - My next patch in
>> sequence  was to fill out enough of the ARMMCEncode to handle mov r0,
>> #0 and bx lr
>> So might as well as add in Nop generation there as well
>
> It already handles those instructions, as well as the basic arithmetic instructions {ADD, AND, BIC, EOR, ORR, SUB}x{ri,rr,rs}. Check the test/MC/ARM/simple-encoding.ll file for a few examples.
>
> I'm actively working on filling out the rest of the encodings, so you probably won't have to worry about that part of things much for now. Later, there will likely be some ELF-specific fixups/relocations type stuff to fill in, but that'll be comparatively minor, I think.
>
> The MC ARM assembly parser isn't yet far enough along to be very viable for these sorts of testcases. I'd suggest using simple .ll files instead and converting them later. The simple-encoding.ll file above, for example.
>
> -Jim


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?

>
>