* [PATCH,testsuite] Xfail gcc.dg/tree-ssa/stdarg-2.c f15 scans
@ 2015-05-05 9:15 Tom de Vries
2015-05-05 9:50 ` Richard Biener
0 siblings, 1 reply; 2+ messages in thread
From: Tom de Vries @ 2015-05-05 9:15 UTC (permalink / raw)
To: GCC Patches
[-- Attachment #1: Type: text/plain, Size: 1708 bytes --]
Hi,
after checking in the 'postpone expanding va_arg till pass_stdarg' patch series,
some scans related to function f15 in gcc.dg/tree-ssa/stdarg-2.c have started
failing.
[ The committed patch series contained a modification of stdarg-2.c, but that
seems to be not complete and not correct. ]
F.i., for s390 we find at
https://gcc.gnu.org/ml/gcc-testresults/2015-05/msg00507.html:
...
FAIL: gcc.dg/tree-ssa/stdarg-2.c scan-tree-dump stdarg "f15: va_list escapes 0,
needs to save 1 GPR units and 2 FPR units"
...
Furthermore, at https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64950#c11 it was
noted that:
...
Unfortunately, the gcc.dg/tree-ssa/stdarg-2.c part of the patch is wrong: the
test now FAILs on i686-unknown-linux-gnu, i686-apple-darwin, and i386-pc-solaris
with -m64: both dumps (i.e. -m32 and -m64) contain
m32/stdarg-2.c.084t.stdarg:f15: va_list escapes 1, needs to save all GPR units
and all FPR units.
m64/stdarg-2.c.084t.stdarg:f15: va_list escapes 1, needs to save all GPR units
and all FPR units.
...
I've filed two PRs:
- PR66010 '[6 Regression] Missed optimization after inlining va_list parameter'
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66010
- PR66013 'Missed optimization after inlining va_list parameter, -m32 case'
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66013
to track these errors.
AFAIU now from the investigations in those PRs, we can expect all f15 scans that
check for the presence of 'va_list escapes 0' to fail.
I'd like to commit two patches.
- The first patch undoes the modification of stdarg-2.c as committed in the
original patch series (omitted).
- The second patch adds appropriate xfails (attached).
OK for trunk?
Thanks,
- Tom
[-- Attachment #2: 0002-Xfail-gcc.dg-tree-ssa-stdarg-2.c-f15-scans.patch --]
[-- Type: text/x-patch, Size: 2518 bytes --]
[PATCH 2/2] Xfail gcc.dg/tree-ssa/stdarg-2.c f15 scans
2015-05-05 Tom de Vries <tom@codesourcery.com>
* gcc.dg/tree-ssa/stdarg-2.c: Xfail f15 scans which test for presence of
'va_list escapes 0'.
---
gcc/testsuite/gcc.dg/tree-ssa/stdarg-2.c | 14 +++++++++++---
1 file changed, 11 insertions(+), 3 deletions(-)
diff --git a/gcc/testsuite/gcc.dg/tree-ssa/stdarg-2.c b/gcc/testsuite/gcc.dg/tree-ssa/stdarg-2.c
index fe39da3..f09b5de 100644
--- a/gcc/testsuite/gcc.dg/tree-ssa/stdarg-2.c
+++ b/gcc/testsuite/gcc.dg/tree-ssa/stdarg-2.c
@@ -288,10 +288,18 @@ f15 (int i, ...)
f15_1 (ap);
va_end (ap);
}
-/* { dg-final { scan-tree-dump "f15: va_list escapes 0, needs to save \[148\] GPR units and \[1-9\]\[0-9\]* FPR units" "stdarg" { target { { i?86-*-* x86_64-*-* } && { ! { ia32 || llp64 } } } } } } */
-/* { dg-final { scan-tree-dump "f15: va_list escapes 0, needs to save \[148\] GPR units and \[1-9\]\[0-9\]* FPR units" "stdarg" { target { powerpc*-*-linux* && { powerpc_fprs && ilp32 } } } } } */
+
+/* Following three dg-finals are marked as xfail due to PR66010/PR66013. */
+/* Was: { target { { i?86-*-* x86_64-*-* } && { ! { ia32 || llp64 } } } }. */
+/* { dg-final { scan-tree-dump "f15: va_list escapes 0, needs to save \[148\] GPR units and \[1-9\]\[0-9\]* FPR units" "stdarg" { xfail *-*-* } } } */
+/* Was: { target { powerpc*-*-linux* && { powerpc_fprs && ilp32 } } }. */
+/* { dg-final { scan-tree-dump "f15: va_list escapes 0, needs to save \[148\] GPR units and \[1-9\]\[0-9\]* FPR units" "stdarg" { xfail *-*-* } } } */
+/* Was: { target s390*-*-linux* }. */
+/* { dg-final { scan-tree-dump "f15: va_list escapes 0, needs to save 1 GPR units and 2 FPR units" "stdarg" { xfail *-*-* } } } */
+
+/* We may be able to improve upon this after fixing PR66010/PR66013. */
/* { dg-final { scan-tree-dump "f15: va_list escapes 1, needs to save all GPR units and all FPR units" "stdarg" { target alpha*-*-linux* } } } */
-/* { dg-final { scan-tree-dump "f15: va_list escapes 0, needs to save 1 GPR units and 2 FPR units" "stdarg" { target s390*-*-linux* } } } */
+
/* { dg-final { scan-tree-dump-not "f15: va_list escapes 0, needs to save 0 GPR units" "stdarg" { target { { i?86-*-* x86_64-*-* } && ia32 } } } } */
/* { dg-final { scan-tree-dump-not "f15: va_list escapes 0, needs to save 0 GPR units" "stdarg" { target ia64-*-* } } } */
/* { dg-final { scan-tree-dump-not "f15: va_list escapes 0, needs to save 0 GPR units" "stdarg" { target { powerpc*-*-* && lp64 } } } } */
--
1.9.1
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH,testsuite] Xfail gcc.dg/tree-ssa/stdarg-2.c f15 scans
2015-05-05 9:15 [PATCH,testsuite] Xfail gcc.dg/tree-ssa/stdarg-2.c f15 scans Tom de Vries
@ 2015-05-05 9:50 ` Richard Biener
0 siblings, 0 replies; 2+ messages in thread
From: Richard Biener @ 2015-05-05 9:50 UTC (permalink / raw)
To: Tom de Vries, GCC Patches
On May 5, 2015 11:15:11 AM GMT+02:00, Tom de Vries <Tom_deVries@mentor.com> wrote:
>Hi,
>
>after checking in the 'postpone expanding va_arg till pass_stdarg'
>patch series,
>some scans related to function f15 in gcc.dg/tree-ssa/stdarg-2.c have
>started
>failing.
>
>[ The committed patch series contained a modification of stdarg-2.c,
>but that
>seems to be not complete and not correct. ]
>
>F.i., for s390 we find at
>https://gcc.gnu.org/ml/gcc-testresults/2015-05/msg00507.html:
>...
>FAIL: gcc.dg/tree-ssa/stdarg-2.c scan-tree-dump stdarg "f15: va_list
>escapes 0,
>needs to save 1 GPR units and 2 FPR units"
>...
>
>Furthermore, at https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64950#c11
>it was
>noted that:
>...
>Unfortunately, the gcc.dg/tree-ssa/stdarg-2.c part of the patch is
>wrong: the
>test now FAILs on i686-unknown-linux-gnu, i686-apple-darwin, and
>i386-pc-solaris
>with -m64: both dumps (i.e. -m32 and -m64) contain
>
>m32/stdarg-2.c.084t.stdarg:f15: va_list escapes 1, needs to save all
>GPR units
>and all FPR units.
>m64/stdarg-2.c.084t.stdarg:f15: va_list escapes 1, needs to save all
>GPR units
>and all FPR units.
>...
>
>I've filed two PRs:
>- PR66010 '[6 Regression] Missed optimization after inlining va_list
>parameter'
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66010
>- PR66013 'Missed optimization after inlining va_list parameter, -m32
>case'
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66013
>to track these errors.
>
>AFAIU now from the investigations in those PRs, we can expect all f15
>scans that
>check for the presence of 'va_list escapes 0' to fail.
>
>I'd like to commit two patches.
>- The first patch undoes the modification of stdarg-2.c as committed in
>the
> original patch series (omitted).
>- The second patch adds appropriate xfails (attached).
>
>OK for trunk?
OK.
Thanks,
Richard.
>Thanks,
>- Tom
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-05-05 9:50 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-05 9:15 [PATCH,testsuite] Xfail gcc.dg/tree-ssa/stdarg-2.c f15 scans Tom de Vries
2015-05-05 9:50 ` Richard Biener
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).