Re: [Patch ARM/AArch64 03/11] AdvSIMD tests: be more verbose.

[Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>]

On 11/05/16 14:23, Christophe Lyon wrote:
> It is useful to have more detailed information in the logs when checking
> validation results: instead of repeating the intrinsic name, we now print
> its return type too.

Ok.
Thanks,
Kyrill

> 2016-05-02  Christophe Lyon  <christophe.lyon@linaro.org>
>
> 	* gcc.target/aarch64/advsimd-intrinsics/arm-neon-ref.h (CHECK,
> 	CHECK_FP, CHECK_CUMULATIVE_SAT): Print which type was checked.
>
> Change-Id: I74759d6a211cf52962f860fe77653a6f6edc1848
>
> diff --git a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/arm-neon-ref.h b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/arm-neon-ref.h
> index 49fbd84..a2c160c 100644
> --- a/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/arm-neon-ref.h
> +++ b/gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/arm-neon-ref.h
> @@ -81,7 +81,7 @@ extern size_t strlen(const char *);
>   	  abort();							\
>   	}								\
>         }									\
> -    fprintf(stderr, "CHECKED %s\n", MSG);				\
> +    fprintf(stderr, "CHECKED %s %s\n", STR(VECT_TYPE(T, W, N)), MSG);	\
>     }
>   
>   /* Floating-point variant.  */
> @@ -110,7 +110,7 @@ extern size_t strlen(const char *);
>   	  abort();							\
>   	}								\
>         }									\
> -    fprintf(stderr, "CHECKED %s\n", MSG);				\
> +    fprintf(stderr, "CHECKED %s %s\n", STR(VECT_TYPE(T, W, N)), MSG);	\
>     }
>   
>   /* Clean buffer with a non-zero pattern to help diagnose buffer
> @@ -335,7 +335,8 @@ extern int VECT_VAR(expected_cumulative_sat, uint, 64, 2);
>   	      strlen(COMMENT) > 0 ? " " COMMENT : "");			\
>         abort();								\
>       }									\
> -    fprintf(stderr, "CHECKED CUMULATIVE SAT %s\n", MSG);		\
> +    fprintf(stderr, "CHECKED CUMULATIVE SAT %s %s\n",			\
> +	    STR(VECT_TYPE(T, W, N)), MSG);				\
>     }
>   
>   #define CHECK_CUMULATIVE_SAT_NAMED(test_name,EXPECTED,comment)		\