================ @@ -2288,7 +2288,9 @@ class Preprocessor { /// Determine whether the next preprocessor token to be /// lexed is a '('. If so, consume the token and return true, if not, this /// method should have no observable side-effect on the lexed tokens. - bool isNextPPTokenLParen(); + bool isNextPPTokenLParen() { + return peekNextPPToken().value_or(Token{}).is(tok::l_paren); ---------------- AaronBallman wrote:
The lexer is almost always on the hot path but `Token`s are small... I think it may be worth verifying that this has no performance difference from an approach without `value_or`. https://github.com/llvm/llvm-project/pull/143898 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits