public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH] Avoid ssa-fre-4.c FAIL on many arches (PR tree-optimization/33757)
@ 2007-10-29 15:49 Jakub Jelinek
  2007-10-29 15:58 ` Richard Guenther
  0 siblings, 1 reply; 2+ messages in thread
From: Jakub Jelinek @ 2007-10-29 15:49 UTC (permalink / raw)
  To: Richard Guenther; +Cc: gcc-patches

Hi!

This test assumes TARGET_PROMOTE_PROTOTYPES returns true, but that's not the
case on many targets.
Rather than adding e.g.
s390*-*-* ia64-*-* alpha*-*-* avr*-*-* c4x*-*-* crx*-*-* frv*-*-*
powerpc*-*-* spu*-*-* m32c*-*-*
sparc*-*-* && lp64
some arm*-*-*
some sh*-*-* (hitachi or renesas types)
to the dg-skip-if, it is IMHO better just to run the test on selected
few arches which are known to TARGET_PROMOTE_PROTOTYPES true.
See various gcc-testresults mails for these failures.

Ok for trunk?

2007-10-29  Jakub Jelinek  <jakub@redhat.com>

	PR tree-optimization/33757
	* gcc.dg/tree-ssa/ssa-fre-4.c: Run on selected TARGET_PROMOTE_PROTOTYPES
	true architectures instead of having a big dg-skip-if.

--- gcc/testsuite/gcc.dg/tree-ssa/ssa-fre-4.c.jj	2007-09-11 12:43:31.000000000 +0200
+++ gcc/testsuite/gcc.dg/tree-ssa/ssa-fre-4.c	2007-10-29 15:36:36.000000000 +0100
@@ -1,7 +1,6 @@
-/* { dg-do compile } */
 /* If the target returns false for TARGET_PROMOTE_PROTOTYPES, then there
    will be no casts for FRE to eliminate and the test will fail.  */
-/* { dg-skip-if "no promotion to eliminate" { cris-*-* mmix-*-* } { "*" } { "" } } */
+/* { dg-do compile { target i?86-*-* x86_64-*-* hppa*-*-* mips*-*-* m68k*-*-* } } */
 /* { dg-options "-O -fdump-tree-fre-details" } */
 
 /* From PR21608.  */

	Jakub

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] Avoid ssa-fre-4.c FAIL on many arches (PR tree-optimization/33757)
  2007-10-29 15:49 [PATCH] Avoid ssa-fre-4.c FAIL on many arches (PR tree-optimization/33757) Jakub Jelinek
@ 2007-10-29 15:58 ` Richard Guenther
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Guenther @ 2007-10-29 15:58 UTC (permalink / raw)
  To: Jakub Jelinek; +Cc: gcc-patches

On Mon, 29 Oct 2007, Jakub Jelinek wrote:

> Hi!
> 
> This test assumes TARGET_PROMOTE_PROTOTYPES returns true, but that's not the
> case on many targets.
> Rather than adding e.g.
> s390*-*-* ia64-*-* alpha*-*-* avr*-*-* c4x*-*-* crx*-*-* frv*-*-*
> powerpc*-*-* spu*-*-* m32c*-*-*
> sparc*-*-* && lp64
> some arm*-*-*
> some sh*-*-* (hitachi or renesas types)
> to the dg-skip-if, it is IMHO better just to run the test on selected
> few arches which are known to TARGET_PROMOTE_PROTOTYPES true.
> See various gcc-testresults mails for these failures.
> 
> Ok for trunk?

Ok.

Thanks,
Richard.

> 
> 2007-10-29  Jakub Jelinek  <jakub@redhat.com>
> 
> 	PR tree-optimization/33757
> 	* gcc.dg/tree-ssa/ssa-fre-4.c: Run on selected TARGET_PROMOTE_PROTOTYPES
> 	true architectures instead of having a big dg-skip-if.
> 
> --- gcc/testsuite/gcc.dg/tree-ssa/ssa-fre-4.c.jj	2007-09-11 12:43:31.000000000 +0200
> +++ gcc/testsuite/gcc.dg/tree-ssa/ssa-fre-4.c	2007-10-29 15:36:36.000000000 +0100
> @@ -1,7 +1,6 @@
> -/* { dg-do compile } */
>  /* If the target returns false for TARGET_PROMOTE_PROTOTYPES, then there
>     will be no casts for FRE to eliminate and the test will fail.  */
> -/* { dg-skip-if "no promotion to eliminate" { cris-*-* mmix-*-* } { "*" } { "" } } */
> +/* { dg-do compile { target i?86-*-* x86_64-*-* hppa*-*-* mips*-*-* m68k*-*-* } } */
>  /* { dg-options "-O -fdump-tree-fre-details" } */
>  
>  /* From PR21608.  */
> 
> 	Jakub
> 
> 

-- 
Richard Guenther <rguenther@suse.de>
Novell / SUSE Labs
SUSE LINUX Products GmbH - Nuernberg - AG Nuernberg - HRB 16746 - GF: Markus Rex

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2007-10-29 15:04 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-10-29 15:49 [PATCH] Avoid ssa-fre-4.c FAIL on many arches (PR tree-optimization/33757) Jakub Jelinek
2007-10-29 15:58 ` Richard Guenther

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).