Re: Potential fix for rdar://4658012

[Josh Conner <jconner at apple dot com>]

Richard Guenther wrote:
> 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 -

I wasn't able to reproduce on i686-pc-linux-gnu.  Do you have any ideas
what might be different?  Where is the ICE occurring?

Thanks -

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());
>> > + }
>> >
>>
>>
>