Re: Potential fix for rdar://4658012

["Richard Guenther" <richard dot guenther at gmail dot com>]

On 8/28/06, Josh Conner <jconner@apple.com> wrote:
> Richard Guenther wrote:
> > On 8/26/06, Richard Guenther <richard.guenther@gmail.com> wrote:
> >> On 26 Aug 2006 11:12:03 -0700, Ian Lance Taylor <iant@google.com> wrote:
> >> > "Richard Guenther" <richard.guenther@gmail.com> writes:
> >> >
> >> > > Instead the following patch makes the C testcase above share the
> >> stack
> >> > > slot.  (completely untested)
>
> Richard -
>
> Thanks for looking into this, and creating the test case.  Could you
> tell me in what way it fails, and on what platform?  I don't see
> incorrect behavior reading through the assembly that is generated, at
> least for ppc, and it at least compiles for linux-x86.

It ICEd for me on i686-pc-linux-gnu.  Unfortunately it doesn't at
work, but only at
home(!?).

Richard.

> - Josh
>
> > Instead it fails for the testcase below.  Committed as obvious.
> >
> > Richard.
> >
> > Index: testsuite/ChangeLog
> > ===================================================================
> > *** testsuite/ChangeLog (revision 116476)
> > --- testsuite/ChangeLog (working copy)
> > ***************
> > *** 1,3 ****
> > --- 1,7 ----
> > + 2006-08-26  Richard Guenther  <rguenther@suse.de>
> > +
> > +       * gcc.c-torture/compile/20060826-1.c: New testcase.
> > +
> >  2006-08-26  Lee Millward  <lee.millward@codesourcery.com>
> >
> >          PR c++/28736
> > Index: testsuite/gcc.c-torture/compile/20060826-1.c
> > ===================================================================
> > *** testsuite/gcc.c-torture/compile/20060826-1.c        (revision 0)
> > --- testsuite/gcc.c-torture/compile/20060826-1.c        (revision 0)
> > ***************
> > *** 0 ****
> > --- 1,7 ----
> > + typedef _Complex double S;
> > + S bar (void);
> > + void quux (S, S);
> > + void foo (void)
> > + {
> > +  quux (bar(), bar());
> > + }
> >