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

Reply via email to