Hi, > --- a/gcc/calls.c > +++ b/gcc/calls.c > @@ -1557,7 +1557,10 @@ maybe_warn_nonstring_arg (tree fndecl, tree exp) > tree bound = NULL_TREE; > > /* The range of lengths of a string argument to one of the comparison > - functions. If the length is less than the bound it is used instead. */ > + functions. If the length is less than the bound it is used instead. > + Since the lengths are only used for warning and not for code > + generation disable strict mode in the calls to get_range_strlen > + below. */ > tree lenrng[2] = { NULL_TREE, NULL_TREE }; > > /* It's safe to call "bounded" string functions with a non-string > @@ -1582,7 +1585,8 @@ maybe_warn_nonstring_arg (tree fndecl, tree exp) > { > tree arg = CALL_EXPR_ARG (exp, argno); > g if (!get_attr_nonstring_decl (arg))
malformed patch here. In general I would prefer to keep in and out parameters separate, but that is probably only personal taste. In the part 2/4 and 3/4 there is a second patch as an attachment, They look like duplicated, but not exactly. I can't follow which one is to be reviewed. Bernd.