[llvm-commits] ARM .fpu attribute

[Jason Kim]

HI Renato,

Patch looks good. Please commit.

I am unsure about the lowercase/uppercase difference between .s and .o
for textual attributes.

AFAIK, it is required for .cpu attribute to be lowercase in .s and
uppercase in .o.
Is it also required (or at least harmless) for .fpu attr as well?
Thanks!

-jason

On Mon, Feb 28, 2011 at 5:34 AM, Renato Golin <Renato.Golin at arm.com> wrote:
> Hi all,
>
> This is a patch to fig bug:
>
> http://llvm.org/bugs/show_bug.cgi?id=8931
>
> Rationale:
>
> The ARM asm backend wasn't printing the .fpu attribute when NEON was
> available (of VFP for that matter) and the linker was getting confused
> on which instructions were available or not.
>
> This patch fix this by printing the .fpu attribute whenever in assembly
> mode and when VFP/NEON is available, exporting both numeric and text
> versions.
>
> thanks,
> --renato
>
> -- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium.  Thank you.