Hi, here is the followup patch. Ok for trunk?
gcc/c-family/
* c-common.h (inv_list): Remove.Thanks, Julia > -----Original Message----- > From: Jeff Law [mailto:[email protected]] > Sent: Monday, November 27, 2017 6:50 PM > To: Koval, Julia <[email protected]>; Joseph Myers > <[email protected]> > Cc: Jakub Jelinek <[email protected]>; GCC Patches <gcc- > [email protected]> > Subject: Re: [patch] remove cilk-plus > > On 11/23/2017 02:45 AM, Koval, Julia wrote: > > Sorry, I think in this version of this patch they are fixed. > > > >> -----Original Message----- > >> From: Joseph Myers [mailto:[email protected]] > >> Sent: Wednesday, November 22, 2017 6:23 PM > >> To: Koval, Julia <[email protected]> > >> Cc: Jeff Law <[email protected]>; Jakub Jelinek <[email protected]>; GCC > >> Patches <[email protected]> > >> Subject: RE: [patch] remove cilk-plus > >> > >> This patch version does not appear to address my comment that you're > >> leaving behind comments in c-parser.c relating to Cilk array notations > >> while removing the subsequent code. (Or in one case actually reindenting > >> the comment that is no longer relevant, rather than removing it.) > >> > >> -- > >> Joseph S. Myers > >> [email protected] > This version is fine to commit. > > Can you also remove struct inv_list from c-family/c-common.h. It was > only used for array notation support. You can include that in your main > commit or as a separate follow-up. > > We may well find other nits like inv_list. I'm comfortable addressing > those as we bump into them. > > Thanks for taking care of this! > > jeff
0001-cilk-followup.patch
Description: 0001-cilk-followup.patch
