-
Notifications
You must be signed in to change notification settings - Fork 363
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
C++ grammar refactoring #593
Comments
From @Bertk: By the way, the C++ standard (n4527) defines
but we use :
|
There are some lines which shall be checked as well - I feel the b.optional("::") should be removed:
|
@Bertk there are also some issues from you solved. Maybe you can also test it? |
I did a small test and it is OK. Did anybody check the performance impact? I got the impression the analysis takes some minutes more but this might also related to other modifications. |
Align grammar with latest C++ standard.
e.g. @Bertk By the way, the sequence of the statements within postfixExpression is not correct and the definition for
postfix-expression [ braced-init-list ]
is missing. New parser errors occurred after moving primary-expression to the binginning of the rule and I stopped this activity.Excerpt from N4527 Annex A Grammar summary:
The text was updated successfully, but these errors were encountered: