This is the mail archive of the
mailing list for the GCC project.
- From: Roger Sayle <roger at eyesopen dot com>
- To: Kaz Kojima <kkojima at rr dot iij4u dot or dot jp>, Joern RENNECKE <joern dot rennecke at st dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Tue, 4 Jan 2005 05:12:37 -0700 (MST)
- Subject: Re: Ping^2
On Tue, 4 Jan 2005, Joern RENNECKE wrote:
> Kaz Kojima wrote:
> >+#if defined (MODE_ENTRY) && defined (MODE_EXIT)
> >+/* Split the edge from the entry block and the fallthrough edge to the
> >+ exit block, so that we can note that there NORMAL_MODE is supplied /
> >+ required. Return the new block if it's inserted before the exit
> >+ block. Otherwise return null. */
> That comment has to be split. Where post_entry is set, you need:
> /* Split the edge from the entry block, so that we can note that
> there NORMAL_MODE is supplied. */
> And for the new function:
> /* Split the fallthrough edge to the exit block, so that we can note
> that there NORMAL_MODE is required. Return the new block if it's
> inserted before the exit block. Otherwise return null. */
> Otherwise, that patch is fine with me.
As Joern's happy with the revised patch, it's OK for mainline with
the comment he's changes suggested above. But don't forget to list
PR target/16482 in the ChangeLog entry.
I'll leave it to the folks actually writing the patches to decide
whether they want to commit this fix now and implement Joern's larger
"consolidation" clean-up as a follow-up patch, or to instead prepare
a "take 3".
Thanks again to you both,