Re: Fix a bug in merging uninitialized refs into a single web

[Jie Zhang <jie at codesourcery dot com>]

On 01/25/2011 08:18 PM, Richard Guenther wrote:
> On Tue, Jan 25, 2011 at 1:15 PM, Jie Zhang<jie@codesourcery.com>  wrote:
> > On 01/25/2011 08:08 PM, Richard Guenther wrote:
> > > On Tue, Jan 25, 2011 at 4:43 AM, Jie Zhang<jie@codesourcery.com>    wrote:
> > > > Hi Richard,
> > > >
> > > > You approved Alexandre's original patch. Could you please also approve my
> > > > patch which fixes a bug in that patch? Thanks!
> > >
> > > Did I? ...
> > Yes, you did. See
> >
> > http://gcc.gnu.org/ml/gcc-patches/2010-01/msg00372.html
> >
> > > Well, I'm not familiar with the code, so I'll defer to somebody else.
> > Alexandre has already confirmed that my fix is OK. But it seems he could not
> > approve it since his original patch was approved by you. So I thought you
> > must be familiar with the code and could also approve mine.
>
> Heh, I relied on Steven here.  As Jeff already had a look maybe he
> can approve your patch.
I asked Jeff, but there was no reply. So I thought the quickest way to 
get my patch approved might be asking you.

> > > > On 01/17/2011 11:05 PM, Alexandre Oliva wrote:
> > > > > On Jan 10, 2011, Jie Zhang<jie@codesourcery.com>      wrote:
> > > > >
> > > > > > Since you are the author of the code my patch changes, could you
> > > > > > please take a look at my patch and confirm that it does the right
> > > > > > thing.
> > > > >
> > > > > Yeah, now that I looked again at my patch from back then, yours makes
> > > > > perfect sense to me.  Thanks!


Regards,
Jie