[patch] Remove superfluous call to fold

[Richard Guenther]

On 8/16/05, Volker Reichelt <reichelt@igpm.rwth-aachen.de> wrote:
> Calling fold on the return value of build_function_call_expr like in
> expand_builtin_strcat in builtins.c
> 
>   newdst =
>     fold (build_function_call_expr (strlen_fn,
>                                     build_tree_list (NULL_TREE,
>                                                      dst)));
> 
> only wastes cycles, since build_function_call_expr already performed
> the folding:
> 
>   tree
>   build_function_call_expr (tree fn, tree arglist)

Following precedence and to avoid such issue in future should we rename
this to fold_build_function_call_expr?  Or build_fold_function_call_expr.

RIchard.