This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] fix #69405 - [6 Regression] ICE in c_tree_printer on an invalid __atomic_fetch_add
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Jeff Law <law at redhat dot com>
- Cc: Martin Sebor <msebor at gmail dot com>, Gcc Patch List <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 21 Jan 2016 22:08:05 +0100
- Subject: Re: [PATCH] fix #69405 - [6 Regression] ICE in c_tree_printer on an invalid __atomic_fetch_add
- Authentication-results: sourceware.org; auth=none
- References: <56A065DE dot 8090002 at gmail dot com> <56A1176B dot 5050604 at redhat dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Thu, Jan 21, 2016 at 10:37:47AM -0700, Jeff Law wrote:
> On 01/20/2016 10:00 PM, Martin Sebor wrote:
> >The attached patch avoids printing a diagnostic referencing the type
> >of an incompatible argument to the __atomic_xxx built-ins when the
> >argument is in error. Doing otherwise causes an ICE as pointed out
> >in the bug, for both of which I am to blame.
> >
> >Martin
> >
> >gcc-69405.patch
> >
> >
> >gcc/testsuite/ChangeLog:
> >2016-01-20 Martin Sebor<msebor@redhat.com>
> >
> > PR c/69405
> > * gcc.dg/sync-fetch.c: New test.
> >
> >gcc/c-family/ChangeLog:
> >2016-01-20 Martin Sebor<msebor@redhat.com>
> >
> > PR c/69405
> > * c-common.c (sync_resolve_size): Avoid printing diagnostic about
> > an incompatible argument when the argument isn't a valid a tree
> > node.
> OK.
Note, the last "a" should go, "a valid tree node". Furthermore, it seems
the last two lines of the c-family/ ChangeLog entry are not tab indented.
Jakub