[PATCH] D33205: ARM] Fix Neon vector type alignment to 64-bit

[James Molloy via llvm-commits]

Hi Javed,

Why reduce the preferred alignment from 128? Natural vector alignment means
we can more often use the :128 hints on loads and stores.

What's the rationale here?

James
On Mon, 15 May 2017 at 18:10, Javed Absar via Phabricator via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> javed.absar created this revision.
> Herald added subscribers: srhines, aemerson.
>
> The maximum alignment for ARM NEON data types should be 64-bits as
> specified in  ARM procedure call standard document Sec. A.2 Notes  [1].
> This patch fixes it from its current larger natural default values.
>
> [1]
> http://infocenter.arm.com/help/topic/com.arm.doc.ihi0042f/IHI0042F_aapcs.pdf
>
>
> https://reviews.llvm.org/D33205
>
> Files:
>   lib/Basic/Targets.cpp
>   test/CodeGen/arm-abi-vector.c
>   test/CodeGen/arm-neon-misc.c
>   test/CodeGen/arm-swiftcall.c
>   test/CodeGen/armv7k-abi.c
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170515/91a35cd0/attachment.html>