On 11/04/2011 04:56 PM, Jason Merrill wrote:
On 11/04/2011 04:14 PM, Richard Henderson wrote:
On 11/04/2011 12:26 PM, Jason Merrill wrote:
On 11/04/2011 03:23 PM, Richard Henderson wrote:
(cp_parser_transaction_expression): Don't parse txn-attributes here.

There's also the issue that the grammar uses compound-statement,
whereas the code parses a parenthesized expression.

There are two different items, actually:

__transaction_* { } // transaction statement, can be aborted
__transaction_* (expr) // transaction expression, cannot, and returns
the value

but we really do require the ( in the second case, so the grammar
shouldn't
require any lookahead.

Right, my point is that the comment before
cp_parser_transaction_expression refers to compound_statement, which
seems wrong.

...because the compound-statement variant is handled in cp_parser_transaction.

Jason

Reply via email to