Re: PR 8134: C++ crash

[Mark Mitchell <mark@codesourcery.com>]

--On Tuesday, October 15, 2002 07:48:06 PM +0100 Nathan Sidwell 
<nathan@codesourcery.com> wrote:

> Mark Mitchell wrote:
>> PR 8134 is a crash in force_store_init_value on the branch; it is a
>> regression.
>
>> a) Nothing
>> b) Move my changes over.
>> c) Revert Alexandre's patch.

Jason wishes there was another choice, but thus far hasn't been able
to come up with one.

Gaby and Benjamin think that my large patch is bug-free.

Nathan isn't so optimistic.

I'm not so optimistic either.  I know my new patch is right in concept,
but I'm not sure it's bugless.  I don't think it's prudent to move over
a patch that substantial when it hasn't received more than a week or two
of testing.

So, I'm going to go with (c).  I'll be updating the caveats HTML page
with information about this.  The good news is that pointers-to-members
are rare, programs relying on zero-initialization of them are rarer,
and there is always a work-around -- explicitly initialize the
pointer-to-member.

I hope people use pointers-to-members a lot on IPF.  The whole deal here
is that we're trying to save an addition per use of a pointer to data
member on a machine that doesn't have base+offset addressing.  I have
this bad feeling that I've spent more time implementing this stuff than
will ever be saved by all the programs in the world running for all time
on architectures where this will help...)

-- 
Mark Mitchell                mark@codesourcery.com
CodeSourcery, LLC            http://www.codesourcery.com