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

Jason Kim jasonwkim at google.com
Wed Oct 6 08:52:50 PDT 2010


Hi everyone.

Can I commit this? If not, some feedback would be great. Thanks!

-jason

On Mon, Oct 4, 2010 at 5:52 PM, Jason Kim <jasonwkim at google.com> wrote:
> Instead of adding a  ARM subclass for MCELFStreamer,  I instead use
> ARMAsmPrinter class.
> Plan is to flush out emitELFAttributes() to emit the blobs in the
> .ARM.attributes section.
> That way, the arch specific cruft is remains in the arch-specific class.
>
> (existing test covers this patch),
>
> Feedback greatly appreciated.
> Diffview:  http://codereview.chromium.org/3598009
>
> Thanks for reading!
>
> -Jason
>




More information about the llvm-commits mailing list