On 4 April 2012 19:35, Matt Turner <matts...@gmail.com> wrote:

>
>  gcc/testsuite/gcc.target/arm/mmx-2.c |  158 
> ++++++++++++++++++++++++++++++++++
>  1 files changed, 158 insertions(+), 0 deletions(-)
>  create mode 100644 gcc/testsuite/gcc.target/arm/mmx-2.c
>
> diff --git a/gcc/testsuite/gcc.target/arm/mmx-2.c 
> b/gcc/testsuite/gcc.target/arm/mmx-2.c
> new file mode 100644
> index 0000000..603a63b
> --- /dev/null
> +++ b/gcc/testsuite/gcc.target/arm/mmx-2.c
> @@ -0,0 +1,158 @@
> +/* { dg-do compile } */
> +/* { dg-skip-if "Test is specific to the iWMMXt" { arm*-*-* } { "-mcpu=*" } 
> { "-mcpu=iwmmxt" } } */
> +/* { dg-skip-if "Test is specific to the iWMMXt" { arm*-*-* } { "-mabi=*" } 
> { "-mabi=iwmmxt" } } */
> +/* { dg-skip-if "Test is specific to the iWMMXt" { arm*-*-* } { "-march=*" } 
> { "-march=iwmmxt" } } */
> +/* { dg-skip-if "Test is specific to ARM mode" { arm*-*-* } { "-mthumb" } { 
> "" } } */

How about simplifying this with a dg-require-effective-target
arm_arm_ok instead of doing
dg-require-effective-target arm32 and then skipping it for Thumb2 ?

Otherwise looks OK to me. Please don't commit any of these patches
till your copyright assignment is sorted -

Thanks,
Ramana

Reply via email to