https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97860
Jakub Jelinek changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97860
--- Comment #8 from CVS Commits ---
The master branch has been updated by Jakub Jelinek :
https://gcc.gnu.org/g:8156cfaa4c45f1249bbdda29d04b4fef84b7eafe
commit r11-5180-g8156cfaa4c45f1249bbdda29d04b4fef84b7eafe
Author: Jakub Jelinek
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97860
--- Comment #7 from Martin Sebor ---
What I mean is that unless error_mark_node necessarily implies (and guarantees)
the bound is a constant zero (as opposed to a similarly "broken" VLA bound),
simply bailing is safer than skipping it. I have
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97860
--- Comment #6 from Jakub Jelinek ---
As I said, [0] is not a VLA bound.
And we don't record anything for constant bounds (even if they are in the
middle).
So perhaps:
/* array_type_nelts assumes the middle-end TYPE_DOMAINs, while
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97860
--- Comment #5 from Martin Sebor ---
Actually, I missed that your patch just skips over the error node. That will
leave the attribute spec out of sync with the argument (it contains a '$' for
each VLA bound). Rather than continuing to the next
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97860
--- Comment #4 from Jakub Jelinek ---
(In reply to Martin Sebor from comment #3)
> I was going to commit the following but I'll leave it to you.
>
> diff --git a/gcc/c/c-decl.c b/gcc/c/c-decl.c
> index d348e39c27a..95cf9e4cb00 100644
> ---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97860
--- Comment #3 from Martin Sebor ---
I was going to commit the following but I'll leave it to you.
diff --git a/gcc/c/c-decl.c b/gcc/c/c-decl.c
index d348e39c27a..95cf9e4cb00 100644
--- a/gcc/c/c-decl.c
+++ b/gcc/c/c-decl.c
@@ -5775,6 +5775,10
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97860
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment #2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97860
Richard Biener changed:
What|Removed |Added
Priority|P3 |P1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97860
Martin Liška changed:
What|Removed |Added
Known to work||10.2.0
Target Milestone|---
10 matches
Mail list logo