Re: [PATCH] Low-hanging C++-lexer speedup (PR c++/24208)

2016-01-27 Thread Patrick Palka
On Wed, Jan 27, 2016 at 4:33 PM, Jason Merrill wrote: > On 01/25/2016 11:45 PM, Patrick Palka wrote: >> >> +/* This needs to be set to TRUE before the lexer-debugging infrastructure >> can >> + be used. The point of this flag is to help the compiler to fold away >> calls >> +

Re: [PATCH] Low-hanging C++-lexer speedup (PR c++/24208)

2016-01-27 Thread Jason Merrill
On 01/25/2016 11:45 PM, Patrick Palka wrote: +/* This needs to be set to TRUE before the lexer-debugging infrastructure can + be used. The point of this flag is to help the compiler to fold away calls + to cp_lexer_debugging_p within this source file at compile time, when the + lexer is

[PATCH] Low-hanging C++-lexer speedup (PR c++/24208)

2016-01-25 Thread Patrick Palka
Within cp/parser.c, cp_lexer_peek_token and the rest of the token-related functions are bloated by lexer-debugging code, code that is completely dead unless calls to the functions cp_lexer_[start|stop]_debugging are deliberately inserted somewhere in the parser source code for temporary debugging