Hi,
On 18/05/2018 15:56, Jason Merrill wrote:
I had in mind moving the call to cp_parser_check_condition_declarator
into the block controlled by cp_parser_parse_definitely; this is a
semantic check that should follow the syntactic checks. If there's no
initializer, it doesn't parse as a condition declaration, so we don't
want to complain about it being a semantically invalid condition
declaration.
If we do that we are back to something very, very, similar to what I
posted at the beginning of the thread, right? Therefore, for all the
testcases which don't have an initializer we end-up with *horrible*,
literally *horrible* cascades of errors, plus we ICE on the c++/84588
variants without an initializer. If you like we can do that and defer
the other related issues - strictly speaking c++/84588 would be fixed -
to another new bug.
Paolo.
