Re: [PATCH][AArch64][11/14] Re-layout SIMD builtin types on builtin expansion

2015-08-03 Thread James Greenhalgh
On Fri, Jul 24, 2015 at 09:38:34AM +0100, Kyrill Tkachov wrote: Thanks, here's an updated version. 2015-07-24 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/aarch64/aarch64.c (aarch64_option_valid_attribute_p): Initialize simd builtins if TARGET_SIMD. *

Re: [PATCH][AArch64][11/14] Re-layout SIMD builtin types on builtin expansion

2015-07-24 Thread Kyrill Tkachov
On 22/07/15 10:11, James Greenhalgh wrote: On Tue, Jul 21, 2015 at 05:59:39PM +0100, Kyrill Tkachov wrote: Sorry, here's the correct version, which uses initialized instead of inited in one of the variable names. Some nits below. Kyrill 2015-07-21 Kyrylo Tkachov kyrylo.tkac...@arm.com

Re: [PATCH][AArch64][11/14] Re-layout SIMD builtin types on builtin expansion

2015-07-22 Thread James Greenhalgh
On Tue, Jul 21, 2015 at 05:59:39PM +0100, Kyrill Tkachov wrote: Sorry, here's the correct version, which uses initialized instead of inited in one of the variable names. Some nits below. Kyrill 2015-07-21 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/aarch64/aarch64.c

Re: [PATCH][AArch64][11/14] Re-layout SIMD builtin types on builtin expansion

2015-07-21 Thread Kyrill Tkachov
Sorry, here's the correct version, which uses initialized instead of inited in one of the variable names. Kyrill 2015-07-21 Kyrylo Tkachov kyrylo.tkac...@arm.com * config/aarch64/aarch64.c (aarch64_option_valid_attribute_p): Initialize simd builtins if TARGET_SIMD. *

[PATCH][AArch64][11/14] Re-layout SIMD builtin types on builtin expansion

2015-07-16 Thread Kyrill Tkachov
Hi all, This patch fixes an ICE that I encountered while testing the series. The testcase in the patch ICEs during builtin expansion because the testcase is compiled with +nofp which means the builtin SIMD types are laid out according to the nofp rules, but later when a function tagged with