On 2/6/24 15:45, Marek Polacek wrote:
On Tue, Feb 06, 2024 at 09:37:44PM +0100, Jakub Jelinek wrote:
Hi!
The deducing this patchset added parsing of this specifier to
cp_parser_decl_specifier_seq unconditionally, but in the C++ grammar
this[opt] only appears in the parameter-declaration non-terminal, so
rather than checking in all the callers of cp_parser_decl_specifier_seq
except for cp_parser_parameter_declaration that this specifier didn't
appear I think it is far easier and closer to what the standard says
to only parse this specifier when called from
cp_parser_parameter_declaration.
Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
FWIW, the patch looks good to me.
Agreed, OK.
Jason
