[PATCH] [ARM] Emit the build attribtue Tag_CPU_unaligned_access

Renato Golin renato.golin at linaro.org
Tue Oct 7 07:51:58 PDT 2014


Hi Charlie,

Have you used Phabricator?

http://reviews.llvm.org/differential/

It's easier to review patches there than by email.

+    AllowsUnalignedMem = Align == StrictAlign ? false : true;

This could be:

     AllowsUnalignedMem = !(Align == StrictAlign);

Also, the CHECK like could be merged into two:

+; NO-STRICT-ALIGN: .eabi_attribute 34, 1
+; STRICT-ALIGN: .eabi_attribute 34, 0

cheers,
--renato

On 7 October 2014 14:37, Charlie Turner <charlie.turner at arm.com> wrote:
> Hi,
>
>
>
> This patch makes ARMAsmPrinter emit the Tag_CPU_unaligned_access build
> attribute describing whether the target supports unaligned data accesses.
>
>
>
> Please find attached,
>
>
>
> Thanks,
>
> Charlie.
>
>
>
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>



More information about the llvm-commits mailing list