Re: [patch testsuite]: Skip some tests for w64 target

[Kai Tietz <Kai dot Tietz at onevision dot com>]

Hi Joseph,

"Joseph S. Myers" <joseph@codesourcery.com> wrote on 04.02.2009 14:19:02:

> On Wed, 4 Feb 2009, Kai Tietz wrote:
> 
> > Hi,
> > 
> > this patch skip some testsuite tests, which are producing an ICE 
reasoned 
> > by stack overflow. At the moment gcc executables for w64 have a stack 
size 
> > of 8MB set. So some of the tests are failing, because they consume (by 

> > recursions in gimplifier on expression handling) too much stack. Sadly 
the 
> > stack size isn't dynamically as for linux.
> > 
> > 2009-02-04  Kai Tietz  <kai.tietz@onevision.com>
> > 
> >         * g++.dg/eh/cleanup1.C: Skip for target x86_64-*-mingw*
> >         reasoned by stack overflow.
> >         * gcc.c-torture/compile/limits-declparen.c: Likewise.
> >         * gcc.c-torture/compile/limits-exprparen.c: Likewise.
> >         * gcc.c-torture/compile/limits-structnest.c: Likewise.
> 
> I think this would skip the tests for x86_64-*-mingw* *target*, but this 

> is an issue with x86_64-*-mingw* *host*.  I don't know if there's a 
simple 
> way of skipping based on host with only changes to individual tests 
(with 
> changes elsewhere it can be done, see e.g. dg-require-host-local), but 
> skipping based on the target is incorrect here.

I get these errors also for i686 mingw and cygwin cross-compilers for w64. 
So I assume it is more a general issue for all windows based targets. 
Maybe the skip option needs even to be extended. Better indeed would be, 
that the recursion deepth for gimplifier would be lowered, but this is out 
of scope for 4.4.

Cheers,
Kai

|  (\_/)  This is Bunny. Copy and paste Bunny
| (='.'=) into your signature to help him gain
| (")_(") world domination.