[PATCH testsuite, arm] cache fp16 hw effective-target tests

[PATCH testsuite, arm] cache fp16 hw effective-target tests

[Sandra Loosemore]

[-- Attachment #1: Type: text/plain, Size: 706 bytes --]

In some bare-metal environments, the tests for fp16 runtime support fail 
in a way that causes a timeout rather than immediate failure.  (E.g., 
the runtime might provide a do-nothing exception handler that just sits 
in a tight loop and never returns.)  This patch changes the 
effective-target tests for fp16 hardware support to cache the result of 
the test so that we don't have to do this more than once.  I think it 
was probably just an oversight that it wasn't done this way originally, 
since the target is hardly likely to sprout fp16 instruction support 
midway through the test run anyway.  ;-)  Anyway, test results are the 
same with this patch, they just run faster.  OK to commit?

-Sandra

[-- Attachment #2: fp16-hw.log --]
[-- Type: text/x-log, Size: 235 bytes --]

2019-09-13  Sandra Loosemore  <sandra@codesourcery.com>

	gcc/testsuite/
	* lib/target-supports.exp
	(check_effective_target_arm_neon_fp16_hw)
	(check_effective_target_arm_fp16_hw): Use check_runtime
	instead of check_runtime_nocache.

[-- Attachment #3: fp16-hw.patch --]
[-- Type: text/x-patch, Size: 696 bytes --]

Index: gcc/testsuite/lib/target-supports.exp
===================================================================
--- gcc/testsuite/lib/target-supports.exp	(revision 275699)
+++ gcc/testsuite/lib/target-supports.exp	(working copy)
@@ -3909,7 +3909,7 @@ proc check_effective_target_arm_neon_fp1
 	return 0
     }
     global et_arm_neon_fp16_flags
-    check_runtime_nocache arm_neon_fp16_hw {
+    check_runtime arm_neon_fp16_hw {
 	int
 	main (int argc, char **argv)
 	{
@@ -4162,7 +4162,7 @@ proc check_effective_target_arm_fp16_hw
 	return 0
     }
     global et_arm_fp16_flags
-    check_runtime_nocache arm_fp16_hw {
+    check_runtime arm_fp16_hw {
 	int
 	main (int argc, char **argv)
 	{

Re: [PATCH testsuite, arm] cache fp16 hw effective-target tests

[Ramana Radhakrishnan]

On Fri, Sep 13, 2019 at 5:31 PM Sandra Loosemore
<sandra@codesourcery.com> wrote:
>
> In some bare-metal environments, the tests for fp16 runtime support fail
> in a way that causes a timeout rather than immediate failure.  (E.g.,
> the runtime might provide a do-nothing exception handler that just sits
> in a tight loop and never returns.)  This patch changes the
> effective-target tests for fp16 hardware support to cache the result of
> the test so that we don't have to do this more than once.  I think it
> was probably just an oversight that it wasn't done this way originally,
> since the target is hardly likely to sprout fp16 instruction support
> midway through the test run anyway.  ;-)  Anyway, test results are the
> same with this patch, they just run faster.  OK to commit?

Ok, thanks.

Ramana

>
> -Sandra