Patchwork [SH] PR 55146 - Add testcase

login
register
mail settings
Submitter Oleg Endo
Date Feb. 3, 2013, 8:54 p.m.
Message ID <1359924861.2367.145.camel@yam-132-YW-E178-FTW>
Download mbox | patch
Permalink /patch/217807/
State New
Headers show

Comments

Oleg Endo - Feb. 3, 2013, 8:54 p.m.
Hi,

The attached patch adds the testcase as it was originally reported in
the PR.  Although it's not a problem on 4.8, I think it would be better
to have this in the test suite.
Tested with
make -k check-gcc RUNTESTFLAGS="sh.exp=pr55146.c --target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}"

OK for trunk?

Cheers,
Oleg

testsuite/ChangeLog:

	PR target/55146:
	* gcc.target/sh/pr55146.c: New.
Kaz Kojima - Feb. 3, 2013, 11:02 p.m.
Oleg Endo <oleg.endo@t-online.de> wrote:
> The attached patch adds the testcase as it was originally reported in
> the PR.  Although it's not a problem on 4.8, I think it would be better
> to have this in the test suite.
> Tested with
> make -k check-gcc RUNTESTFLAGS="sh.exp=pr55146.c --target_board=sh-sim
> \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}"
> 
> OK for trunk?

OK.

Regards,
	kaz

Patch

Index: gcc/testsuite/gcc.target/sh/pr55146.c
===================================================================
--- gcc/testsuite/gcc.target/sh/pr55146.c	(revision 0)
+++ gcc/testsuite/gcc.target/sh/pr55146.c	(revision 0)
@@ -0,0 +1,50 @@ 
+/* Check that the 'extu.b' instruction is generated for short jump tables.  */
+/* { dg-do compile { target "sh*-*-*" } } */
+/* { dg-options "-Os" } */
+/* { dg-skip-if "" { "sh*-*-*" } { "-m5*"} { "" } }  */
+/* { dg-final { scan-assembler "extu.b" } } */
+
+int
+test (int arg)
+{
+  int rc;
+  switch (arg)
+    {
+    case 0:
+      asm ("nop\n\tnop\n\tnop\n\tnop\n\tnop\n\t"
+	   "nop\n\tnop\n\tnop\n\tnop\n\tnop\n\tnop\n\t"
+	   "mov r4,%0"
+	   : "=r" (rc)
+	   : "r" (arg));
+      break;
+    case 1:
+      asm ("nop\n\tnop\n\tnop\n\tnop\n\tnop\n\t"
+	   "nop\n\tnop\n\tnop\n\tnop\n\tnop\n\tnop\n\t"
+	   "mov r5,%0"
+	   : "=r" (rc)
+	   : "r" (arg));
+      break;
+    case 2:
+      asm ("nop\n\tnop\n\tnop\n\tnop\n\tnop\n\t"
+	   "nop\n\tnop\n\tnop\n\tnop\n\tnop\n\tnop\n\t"
+	   "mov r6,%0"
+	   : "=r" (rc)
+	   : "r" (arg));
+      break;
+    case 3:
+      asm ("nop\n\tnop\n\tnop\n\tnop\n\tnop\n\t"
+	   "nop\n\tnop\n\tnop\n\tnop\n\tnop\n\tnop\n\tnop\n\t"
+	   "mov r7,%0"
+	   : "=r" (rc)
+	   : "r" (arg));
+      break;
+    case 4:
+      asm ("nop\n\tnop\n\tnop\n\tnop\n\tnop\n\t"
+	   "nop\n\tnop\n\tnop\n\tnop\n\tnop\n\tnop\n\tnop\n\t"
+	   "mov r8,%0"
+	   : "=r" (rc)
+	   : "r" (arg));
+      break;
+    }
+  return rc;
+}