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