> 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... Thanks, Igor > > jeff
