On 08/01/2017 04:21 PM, David Malcolm wrote:
@@ -27632,6 +27769,9 @@ cp_parser_sizeof_operand (cp_parser* parser, enum rid 
keyword)
     {
       tree type = NULL_TREE;
+ matching_parens parens;
+      parens.peek_open (parser);

I was puzzled by this until I found that cp_parser_compound_literal_p consumes the open paren. Let's remove that in favor of calling consume_open here, so we don't need peek_open anymore.

About passing parser in or not, I'm happy with the current approach; adding things to the stack isn't free in a highly recursive program like GCC.

Jason

Reply via email to