On 06/19/14 06:30, Zamyatin, Igor wrote:
On 06/16/14 14:13, Zamyatin, Igor wrote:
Hi All!
The patch fixes ICE in array notation for the cases of incorrect arguments of
Cilk+ builtins and undeclared initial index.
Is it ok for trunk and 4.9?
Thanks,
Igor
diff --git a/gcc/c/ChangeLog b/gcc/c/ChangeLog index 54d0de7..56e1b0b
100644
--- a/gcc/c/ChangeLog
+++ b/gcc/c/ChangeLog
@@ -1,3 +1,12 @@
+2014-06-16 Igor Zamyatin <igor.zamya...@intel.com>
+
+ PR middle-end/57541
+ * c-array-notation.c (fix_builtin_array_notation_fn):
+ Check for 0 arguments in builtin call. Check that bultin argument is
+ correct.
+ * c-parser.c (c_parser_array_notation): Check for incorrect initial
+ index.
Shouldn't this have been caught earlier? ISTM we should be catching any
argument mix-ups during parsing?!? Is there some reason we don't do
that?
But call stack for fix_builtin_array_notation_fn is from c-parser...
Joys :( In that case, approved for the trunk & 4.9.
Jeff