Mike Stump wrote:
> On Nov 18, 2011, at 9:04 AM, "Ulrich Weigand" <uweig...@de.ibm.com> wrote:
> > Huh, it seems the test in this form assumes the assembler supports /* ...  
> > */
> > comments.  But this isn't really necessary for the test to work; I think the
> > best fix would be to just remove the comment and simply write 20
> >   __asm__ volatile ("" : : "nro" (e1), "nro" (e2), "nro" (e3 ...
> > instead.
> 
> Sounds good to me.

OK, I've checked in the following patch.

Tested on i386-linux.

Bye,
Ulrich

ChangeLog:

        * gcc.c-torture/compile/pr44707.c: Do not assume assembler
        supports /* ... */ comments.

Index: gcc/testsuite/gcc.c-torture/compile/pr44707.c
===================================================================
--- gcc/testsuite/gcc.c-torture/compile/pr44707.c       (revision 181162)
+++ gcc/testsuite/gcc.c-torture/compile/pr44707.c       (working copy)
@@ -9,6 +9,6 @@
   int e3 = v.b;
   int e4 = v.c;
   int e5 = v.d;
-  __asm__ volatile ("/* %0 %1 %2 %3 %4 */" : : "nro" (e1), "nro" (e2), "nro" 
(e3), "nro" (e4), "nro" (e5));
+  __asm__ volatile ("" : : "nro" (e1), "nro" (e2), "nro" (e3), "nro" (e4), 
"nro" (e5));
 }
 

-- 
  Dr. Ulrich Weigand
  GNU Toolchain for Linux on System z and Cell BE
  ulrich.weig...@de.ibm.com

Reply via email to