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

[Jason Kim]

Forwarding to commits.


---------- Forwarded message ----------
From: Jason Kim <jasonwkim at google.com>
Date: Mon, Oct 25, 2010 at 10:21 AM
Subject: Re: [LLVMdev] Fwd: [llvm-commits] Fwd: Proof of concept patch
for unifying the .s/ELF emission of .ARM.attributes
To: Rafael Espíndola <rafael.espindola at gmail.com>
Cc: LLVM Developers Mailing List <llvmdev at cs.uiuc.edu>, Jim Grosbach
<grosbach at apple.com>, Rafael Espindola <espindola at google.com>


All well and good, Rafael, 3 new classes and 4 new slots.
In any case, I am done arguing this - outside of any additional input
on this might as well as commit your rework. Any objections?

On Sun, Oct 24, 2010 at 7:01 PM, Rafael Espíndola
<rafael.espindola at gmail.com> wrote:
> I also noticed that we were trying to optimize the output of 41 bytes
> of data :-)

Err, Yes. 2+ weeks on this. I am without words at the moment.

> The attached patch is similar to the previous one but drops the API
> changes by just accumulating the attributes locally before outputting
> them.

-jason