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.

Reply via email to