Re: Fix pr50607 bconstp-3.c failure

2011-10-05 Thread Hans-Peter Nilsson
On Thu, 6 Oct 2011, Artem Shinkarov wrote: > On Thu, Oct 6, 2011 at 3:27 AM, Hans-Peter Nilsson wrote: > > Write that changelog entry as: > > > >        PR middle-end/50607 > >        * c-tree.h (c_expr_t): New typedef for struct c_expr. > >        (C_EXPR_APPEND): New macro. > >        * c-parser

Re: Fix pr50607 bconstp-3.c failure

2011-10-05 Thread Artem Shinkarov
On Thu, Oct 6, 2011 at 3:27 AM, Hans-Peter Nilsson wrote: > On Thu, 6 Oct 2011, Artem Shinkarov wrote: >> Successfully regtested on x86-unknown-linux-gnu. Committed to the >> mainline with the revision 179588. >> >> ChangeLog: >> 2011-10-06  Artjoms Sinkarovs   >>         * c-tree.h (c_expr_t): Ne

Re: Fix pr50607 bconstp-3.c failure

2011-10-05 Thread Hans-Peter Nilsson
On Thu, 6 Oct 2011, Artem Shinkarov wrote: > Successfully regtested on x86-unknown-linux-gnu. Committed to the > mainline with the revision 179588. > > ChangeLog: > 2011-10-06 Artjoms Sinkarovs > * c-tree.h (c_expr_t): New typedef for struct c_expr. > (C_EXPR_APPEND): New macro.

Re: Fix pr50607 bconstp-3.c failure

2011-10-05 Thread Artem Shinkarov
On Wed, Oct 5, 2011 at 5:32 PM, Artem Shinkarov wrote: > On Wed, Oct 5, 2011 at 5:28 PM, Joseph S. Myers > wrote: >> On Wed, 5 Oct 2011, Artem Shinkarov wrote: >> >>> Joseph, is it possible to commit the patch together with the space fixes? >> >> You should not commit whitespace fixes to lines n

Re: Fix pr50607 bconstp-3.c failure

2011-10-05 Thread Artem Shinkarov
On Wed, Oct 5, 2011 at 5:28 PM, Joseph S. Myers wrote: > On Wed, 5 Oct 2011, Artem Shinkarov wrote: > >> Joseph, is it possible to commit the patch together with the space fixes? > > You should not commit whitespace fixes to lines not otherwise modified by > a patch, except in a separate commit th

Re: Fix pr50607 bconstp-3.c failure

2011-10-05 Thread Joseph S. Myers
On Wed, 5 Oct 2011, Artem Shinkarov wrote: > Joseph, is it possible to commit the patch together with the space fixes? You should not commit whitespace fixes to lines not otherwise modified by a patch, except in a separate commit that *only* has whitespace or other formatting fixes, so that "sv

Re: Fix pr50607 bconstp-3.c failure

2011-10-05 Thread Artem Shinkarov
On Wed, Oct 5, 2011 at 4:22 PM, Joseph S. Myers wrote: > On Wed, 5 Oct 2011, Artem Shinkarov wrote: > >> On Tue, Oct 4, 2011 at 11:51 PM, Joseph S. Myers >> wrote: >> > On Tue, 4 Oct 2011, Artem Shinkarov wrote: >> > >> >> Hi >> >> >> >> Here is the patch tho fix bconstp-3.c failure in the bug 50

Re: Fix pr50607 bconstp-3.c failure

2011-10-05 Thread Joseph S. Myers
On Wed, 5 Oct 2011, Artem Shinkarov wrote: > On Tue, Oct 4, 2011 at 11:51 PM, Joseph S. Myers > wrote: > > On Tue, 4 Oct 2011, Artem Shinkarov wrote: > > > >> Hi > >> > >> Here is the patch tho fix bconstp-3.c failure in the bug 50607. The > >> failure was cause because the new parser routine did

Re: Fix pr50607 bconstp-3.c failure

2011-10-04 Thread Artem Shinkarov
On Tue, Oct 4, 2011 at 11:51 PM, Joseph S. Myers wrote: > On Tue, 4 Oct 2011, Artem Shinkarov wrote: > >> Hi >> >> Here is the patch tho fix bconstp-3.c failure in the bug 50607. The >> failure was cause because the new parser routine did not consider >> original_tree_code of the expression. >> >>

Re: Fix pr50607 bconstp-3.c failure

2011-10-04 Thread Joseph S. Myers
On Tue, 4 Oct 2011, Artem Shinkarov wrote: > Hi > > Here is the patch tho fix bconstp-3.c failure in the bug 50607. The > failure was cause because the new parser routine did not consider > original_tree_code of the expression. > > The patch is bootstrapped on x86-unknown-linux-gnu and is being

Fix pr50607 bconstp-3.c failure

2011-10-04 Thread Artem Shinkarov
Hi Here is the patch tho fix bconstp-3.c failure in the bug 50607. The failure was cause because the new parser routine did not consider original_tree_code of the expression. The patch is bootstrapped on x86-unknown-linux-gnu and is being tested. Thanks, Artem. Index: c-parser.c ===