On 11-Mar-13, at 5:25 AM, Rainer Orth wrote:

You most likely need whitespace before the xfail.  Also try to add a
comment (PR reference?) explaining it.

Added comments explaining xfails for gcc.dg/tree-ssa/pr55579.c and gcc.dg/tree-ssa/vector-4.c.
Fixed whitespace.

Committed after testing on hppa2.0w-hp-hpux11.11.

Dave
--
John David Anglin       dave.ang...@bell.net


2013-03-11  John David Anglin  <dave.ang...@nrc-cnrc.gc.ca>

        * gcc.dg/tree-ssa/vector-4.c: Add comment regarding xfail.
        * gcc.dg/tree-ssa/pr55579.c: Likewise.

Index: gcc.dg/tree-ssa/vector-4.c
===================================================================
--- gcc.dg/tree-ssa/vector-4.c  (revision 196592)
+++ gcc.dg/tree-ssa/vector-4.c  (working copy)
@@ -9,6 +9,7 @@
 }
 
 /* The compound literal should be placed directly in the vec_perm.  */
+/* Test is xfailed on 32-bit hppa*-*-* because target-callee-copies.  */
 /* { dg-final { scan-tree-dump-times "VEC_PERM_EXPR <a, b, { 0, 4, 1, 5 }>;" 1 
"gimple" { xfail { hppa*-*-* && { ! lp64 } } } } } */
 
 /* { dg-final { cleanup-tree-dump "gimple" } } */
Index: gcc.dg/tree-ssa/pr55579.c
===================================================================
--- gcc.dg/tree-ssa/pr55579.c   (revision 196592)
+++ gcc.dg/tree-ssa/pr55579.c   (working copy)
@@ -11,5 +11,6 @@
   return x;
 }
 
-/* { dg-final { scan-tree-dump "Created a debug-only replacement for s" "esra" 
{xfail { hppa*-*-hpux* && { ! lp64 } } } } } */
+/* Test fails on 32-bit hppa*-*-hpux*.  See PR debug/56307.  */
+/* { dg-final { scan-tree-dump "Created a debug-only replacement for s" "esra" 
{ xfail { hppa*-*-hpux* && { ! lp64 } } } } } */
 /* { dg-final { cleanup-tree-dump "esra" } } */

Reply via email to