On 04/07/13 09:17, James Greenhalgh wrote:
I've committed this as r200634.
As this is a bug-fix I'd like to backport it to the 4.8 branch.
I've attached a copy of the patch that applies to 4.8
and run it through testing on aarch64-none-elf with no issues.
Is this OK to commit to gcc-4_8-branch
Ping?
> -Original Message-
> From: James Greenhalgh
> Sent: 04 July 2013 09:18
> To: gcc-patches@gcc.gnu.org
> Cc: Marcus Shawcroft
> Subject: RE: [AArch64] Convert ld1, st1 arm_neon.h intrinsics to RTL
> builtins.
>
>
> > > 2013-07-02 James Greenha
> > 2013-07-02 James Greenhalgh
> >
> > * config/aarch64/aarch64-builtins.c
> > (aarch64_simd_expand_builtin): Handle AARCH64_SIMD_STORE1.
> > * config/aarch64/aarch64-simd-builtins.def (ld1): New.
> > (st1): Likewise.
> > * config/aarch64/aarch64-simd.md
> > (aarch64_ld
On 02/07/13 10:01, James Greenhalgh wrote:
2013-07-02 James Greenhalgh
* config/aarch64/aarch64-builtins.c
(aarch64_simd_expand_builtin): Handle AARCH64_SIMD_STORE1.
* config/aarch64/aarch64-simd-builtins.def (ld1): New.
(st1): Likewise.
* config/aarch
Hello,
As they stand the ld1 and st1 intrinsics are broken. The constraints
used in their assembler definitions are not strong enough to prevent
GCC from performing some exciting reorders moving around loads and
stores.
Rewriting these intrinsics as RTL builtins settles GCC's errant
behaviour an
