Hi! Just 2 nits:
On Fri, Oct 28, 2016 at 01:46:57PM +0200, Richard Biener wrote: > +/* Return a pointer to the Nth token in PARERs tokens_buf. */ PARSERs ? > @@ -454,7 +423,7 @@ c_lex_one_token (c_parser *parser, c_token *token) > /* Return a pointer to the next token from PARSER, reading it in if > necessary. */ > > -static inline c_token * > +c_token * > c_parser_peek_token (c_parser *parser) > { > if (parser->tokens_avail == 0) I wonder if turning all of these into non-inlines is a good idea. Can't you move them to the common header instead? The rest I defer to Joseph or Marek. Jakub