[PATCH][ARM,AARCH64] target/PR68674: relayout vector_types in expand_expr

Christophe Lyon christophe.lyon@linaro.org
Mon Jan 25 19:06:00 GMT 2016


On 22 January 2016 at 12:56, Richard Biener <richard.guenther@gmail.com> wrote:
> On Fri, Jan 22, 2016 at 12:41 PM, Christian Bruel
> <christian.bruel@st.com> wrote:
>>
>>
>> On 01/19/2016 04:18 PM, Richard Biener wrote:
>>>
>>> maybe just if (currently_expanding_to_rtl)?
>>>
>>> But yes, this looks like a safe variant of the fix.
>>>
>>> Richard.
>>>
>> thanks, currently_expanding_to_rtl works perfectly. So the final version.
>> I added a test for each target.
>
> Ok.
>

Hi,

This small patch is needed to make the new test pass on arm hard-float
targets (eg. arm-none-linux-gnueabihf).

I'm not sure it counts as obvious, so here it is.
OK?

Christophe.

DATE  Christophe Lyon  <christophe.lyon@linaro.org>

    * gcc.target/arm/pr68674.c: Check and use arm_fp effective target.


> Thanks,
> Richard.
>
>> bootstrapped / tested for :
>>     unix/-m32/-march=i586
>>     unix
>>
>>     arm-qemu/
>>     arm-qemu//-mfpu=neon
>>     arm-qemu//-mfpu=neon-fp-armv8
>>
>>     aarch64-qemu
>>
>>
>>
>>
>>
>>
>>
-------------- next part --------------
diff --git a/gcc/testsuite/gcc.target/arm/pr68674.c b/gcc/testsuite/gcc.target/arm/pr68674.c
index a31a88a..0b32374 100644
--- a/gcc/testsuite/gcc.target/arm/pr68674.c
+++ b/gcc/testsuite/gcc.target/arm/pr68674.c
@@ -1,7 +1,9 @@
 /* PR target/68674 */
 /* { dg-do compile } */
 /* { dg-require-effective-target arm_neon_ok } */
-/* { dg-options "-O2 -mfloat-abi=softfp" } */
+/* { dg-require-effective-target arm_fp_ok } */
+/* { dg-options "-O2" } */
+/* { dg-add-options arm_fp } */
 
 #pragma GCC target ("fpu=vfp")
 


More information about the Gcc-patches mailing list