On Mon, Jun 05, 2017 at 08:46:21AM -0400, Nathan Sidwell wrote: > On 06/05/2017 07:47 AM, Jakub Jelinek wrote: > > > Whether there is a blank between function and its function comment > > is something we aren't consistent in, but it seems that in dumpfile.c > > there is a blank line, so I'm adjusting. > > Ok (I only know the blank line rule because someone nitted one of my > patches) > > > > +#define FIRST_AUTO_NUMBERED_DUMP 1 > > +#define FIRST_ME_AUTO_NUMBERED_DUMP 3 > > When you commit, could you set this to 4 (at least). Right now you have no > headroom for C++ FE changes. And I have a branch with a new lang dump :)
It was the intent that there is no unnecessary gap, the difference between those two should be simply the maximum any FE registers. So, on your branch you'd bump it to 4 and on trunk when merging your branch. Jakub