This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [ARM][testsuite] Add missing guards to fp16 AdvSIMD tests

From: Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>
To: Christophe Lyon <christophe dot lyon at linaro dot org>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Date: Mon, 27 Jun 2016 11:59:16 +0100
Subject: Re: [ARM][testsuite] Add missing guards to fp16 AdvSIMD tests
Authentication-results: sourceware.org; auth=none
References: <CAKdteOYKrRVNBrPziSAON6+FdUefu2hed+tzv2cqBqxwHKT+8A at mail dot gmail dot com>


On 21/06/16 09:45, Christophe Lyon wrote:

Hi,

I've noticed that some guards were missing on some of the AdvSIMD
tests involving fp16 code.

The attached patch fixes, although I didn't notice any difference in
validation: I have no configuration where
check_effective_target_arm_neon_fp16_ok fails.

I did locally modify this effective target to always return false to
make sure I covered all the missing guards.

However, I'm not sure when check_effective_target_arm_neon_fp16_ok can
fail? This effective target test was added by Alan Lawrence a few
months ago.

OK?


Ok.
Thanks,
Kyrill

Christophe

References:
- [ARM][testsuite] Add missing guards to fp16 AdvSIMD tests
  - From: Christophe Lyon

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]