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" } } */