[[ARM/AArch64][testsuite] 02/36] Be more verbose, and actually confirm that a test was checked.

Tejas Belagod tejas.belagod@arm.com
Fri Jan 16 13:46:00 GMT 2015


On 13/01/15 15:18, Christophe Lyon wrote:
> 	* gcc.target/aarch64/advsimd-intrinsics/arm-neon-ref.h (CHECK):
> 	Add trace.
> 	(CHECK_FP): Likewise.
> 	(CHECK_CUMULATIVE_SAT): Likewise.
>
> 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 6464c66..2730a66 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
> @@ -79,6 +79,7 @@ extern size_t strlen(const char *);
>   	  abort();							\
>   	}								\
>         }									\
> +    fprintf(stderr, "CHECKED %s\n", MSG);				\
>     }
>
>   /* Floating-point variant.  */
> @@ -107,6 +108,7 @@ extern size_t strlen(const char *);
>   	  abort();							\
>   	}								\
>         }									\
> +    fprintf(stderr, "CHECKED %s\n", MSG);				\
>     }
>
>   /* Clean buffer with a non-zero pattern to help diagnose buffer
> @@ -323,6 +325,7 @@ extern int VECT_VAR(expected_cumulative_sat, uint, 64, 2);
>   	      strlen(COMMENT) > 0 ? " " COMMENT : "");			\
>         abort();								\
>       }									\
> +    fprintf(stderr, "CHECKED CUMULATIVE SAT %s\n", MSG);		\
>     }
>
>   #define CHECK_CUMULATIVE_SAT_NAMED(test_name,EXPECTED,comment)		\
>

Looks OK to me(but I can't approve it).

Tejas.



More information about the Gcc-patches mailing list