On Mon, 13 Dec 2021, Jakub Jelinek via Gcc-patches wrote:

> Hi!
> 
> We ICE on the following testcase, because c_parser_balanced_token_sequence
> when encountering a deferred pragma will just use c_parser_consume_token
> which the FE doesn't allow for CPP_PRAGMA tokens (and if that wasn't
> the case, it could ICE on CPP_PRAGMA_EOL similarly).
> We don't know in what exact context the pragma appears when we don't
> know what those arguments semantically mean, so I think we should just
> skip over them, like e.g. the C++ FE does.  And, I think (/[/{ vs. )/]/}
> from outside of the pragma shouldn't be paired with those inside of
> the pragma and it doesn't seem to be necessary to check that inside of
> the pragma line itself all the paren kinds are balanced.
> 
> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?

OK.

-- 
Joseph S. Myers
jos...@codesourcery.com

Reply via email to