On 29/06/11 09:50, Xinyu Qi wrote:
> Hi,
> 
> This patch adds some simple iWMMXt intrinsic test cases.
> 
> gcc.target/arm/iwmmxt/i_shift_error.c: New.
> gcc.target/arm/iwmmxt/iwmmxt.exp: New.
> gcc.target/arm/iwmmxt/i_wshufh_error.c: New.
> gcc.target/arm/iwmmxt/i_merge_si64_error.c: New.
> gcc.target/arm/iwmmxt/i_waligni_error.c: New.
> gcc.target/arm/iwmmxt/i_extract_error.c: New.
> 

Please use "New test" when adding new tests.

> Thanks,
> Xinyu
> 
> 
> iwmmxt_testsuite.patch
> 
> 
> Index: gcc/testsuite/gcc.target/arm/iwmmxt/i_shift_error.c
> ===================================================================
> --- gcc/testsuite/gcc.target/arm/iwmmxt/i_shift_error.c       (revision 0)
> +++ gcc/testsuite/gcc.target/arm/iwmmxt/i_shift_error.c       (revision 0)
> @@ -0,0 +1,39 @@
> +/* { dg-do compile } */
> +/* { dg-options "-O2 -march=iwmmxt2" } */

No, this will interact badly with multi-lib testing.  You need to add a
test along the lines of arm_iwmmxt_ok from lib/target-supports.exp

R.

Reply via email to