Re: Aliasing patch for tree-optimization/28778

[Mark Mitchell <mark at codesourcery dot com>]

Daniel Berlin wrote:
> On 8/27/06, Mark Mitchell <mark@codesourcery.com> wrote:
> > Daniel Berlin wrote:
> >
> > > I don't disagree, I just don't believe the size of the change it will
> > > take to fix this (probably about 500-1000 lines), and how it is 
> > going to
> > > affect core infrastructure, is really something that should be in 
> > stage3.
> > >
> > > But you are the release manager, so if you want me to write up 
> > something
> > > to fix this correctly, i will do it for 4.2. It will take about 2 weeks
> > > though, as i can't do it during work hours right now (google assignment
> > > issues).
> >
> > I would certainly like you to try to fix the bug.  Then, we can decide
> > whether to take the risk of putting it in during Stage 3, or, at worst,
> > have it ready to go for 4.3.  We might even consider branching, putting
> > it into 4.3, and then backporting it.  But, if we don't have the code,
> > we don't have any options.
>
> I will code up a non-pessimizing correct fix.

Thanks.

> The pessimizing version of the conservative correct fix (IE mark
> everything clobbered in any non-trivial function)  was attached to
> 28778 by Richard Guenther if you want to look at it.

Is this the patch in Comment #20?  (The patch currently attached to the 
PR comes from Daniel, not from Richard.)

Thanks,

--
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713