Hi Thomas,
Am 21.02.23 um 08:19 schrieb Thomas Koenig via Gcc-patches:
Hi Harald,
the attached patch fixes an ICE on invalid (non-integer)
specification expressions for character length in function
declarations. It appears that the error handling was
already in place (mostly) and we need to essentially
prevent run-on errors.
Regtested on x86_64-pc-linux-gnu. OK for mainline?
As a very minor matter of style, you might want to write
function_result_typed = check_function_result_typed ();
instead of
if (check_function_result_typed ())
function_result_typed = true;
I was considering that too, but believed that the logic around
these places (a loop and an if) would confuse readers.
Thinking again and rechecking, I've changed the patch to follow
your suggestion, including a minor style cleanup.
Committed as:
https://gcc.gnu.org/g:6c1b825b3d6499dfeacf7c79dcf4b56a393ac204
commit r13-6265-g6c1b825b3d6499dfeacf7c79dcf4b56a393ac204
Author: Harald Anlauf
Date: Mon Feb 20 21:28:09 2023 +0100
OK either way.
The PR is marked as a 10/11/12/13 regression, so I would
like to backport this as far as it seems reasonable.
Also OK.
Thanks for the patch!
Thanks for the review!
Harald
Best regards
Thomas
