On 22 January 2016 at 12:56, Richard Biener <richard.guent...@gmail.com> wrote:
> On Fri, Jan 22, 2016 at 12:41 PM, Christian Bruel
> <christian.br...@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.l...@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
>>
>>
>>
>>
>>
>>
>>
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")
