* Merge PTR_PLUS branch?
@ 2007-06-14 0:01 Mark Mitchell
2007-06-14 3:33 ` Andrew Pinski
0 siblings, 1 reply; 4+ messages in thread
From: Mark Mitchell @ 2007-06-14 0:01 UTC (permalink / raw)
To: Andrew Pinski; +Cc: GCC
Andrew --
Now that dataflow has gone in, you have the ball. When you feel that
PTR_PLUS is in suitable shape to go in, and the patches have been
reviewed, please merge and commit.
I know that there have been some reviews of the patches along the way,
and the concept has been appropriately approved, so I don't mean for you
to get bogged down in the review process. But, I would like a
maintainer to approve the commit. I will help review if necessary.
Thanks,
--
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Merge PTR_PLUS branch?
2007-06-14 0:01 Merge PTR_PLUS branch? Mark Mitchell
@ 2007-06-14 3:33 ` Andrew Pinski
2007-06-14 4:44 ` Diego Novillo
2007-06-14 16:11 ` Mark Mitchell
0 siblings, 2 replies; 4+ messages in thread
From: Andrew Pinski @ 2007-06-14 3:33 UTC (permalink / raw)
To: Mark Mitchell; +Cc: GCC
On 6/13/07, Mark Mitchell <mark@codesourcery.com> wrote:
> Andrew --
>
> Now that dataflow has gone in, you have the ball. When you feel that
> PTR_PLUS is in suitable shape to go in, and the patches have been
> reviewed, please merge and commit.
Thanks for the confirmation.
> I know that there have been some reviews of the patches along the way,
> and the concept has been appropriately approved, so I don't mean for you
> to get bogged down in the review process. But, I would like a
> maintainer to approve the commit. I will help review if necessary.
I am testing the changes which were requested earlier today. I could
not tell if Diego's was an official approval or not. I have approval
currently officially only for the Fortran parts.
If you could review the C++ front-end changes, that would be nice.
The only difference in the C++ front-end between the patch which I
sent out and my next version is the use of fold_build1 instead of
build1 for the negative expressions.
Thanks,
Andrew Pinski
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Merge PTR_PLUS branch?
2007-06-14 3:33 ` Andrew Pinski
@ 2007-06-14 4:44 ` Diego Novillo
2007-06-14 16:11 ` Mark Mitchell
1 sibling, 0 replies; 4+ messages in thread
From: Diego Novillo @ 2007-06-14 4:44 UTC (permalink / raw)
To: Andrew Pinski; +Cc: Mark Mitchell, GCC
On 6/13/07 11:33 PM, Andrew Pinski wrote:
> I am testing the changes which were requested earlier today. I could
> not tell if Diego's was an official approval or not. I have approval
> currently officially only for the Fortran parts.
It was, with the changes I suggested, for the non-FE parts. Sorry for
not making that clear.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Merge PTR_PLUS branch?
2007-06-14 3:33 ` Andrew Pinski
2007-06-14 4:44 ` Diego Novillo
@ 2007-06-14 16:11 ` Mark Mitchell
1 sibling, 0 replies; 4+ messages in thread
From: Mark Mitchell @ 2007-06-14 16:11 UTC (permalink / raw)
To: Andrew Pinski; +Cc: GCC
Andrew Pinski wrote:
> If you could review the C++ front-end changes, that would be nice.
Would you please point me at a URL for those changes?
Thanks,
--
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2007-06-14 16:11 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-06-14 0:01 Merge PTR_PLUS branch? Mark Mitchell
2007-06-14 3:33 ` Andrew Pinski
2007-06-14 4:44 ` Diego Novillo
2007-06-14 16:11 ` Mark Mitchell
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).