This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] cp/tree.c remove unused function
- From: Eric Christopher <echristo at redhat dot com>
- To: Andreas Tobler <toa at pop dot agri dot ch>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 19 Aug 2004 15:31:55 -0700
- Subject: Re: [patch] cp/tree.c remove unused function
- References: <412529ED.9090707@pop.agri.ch>
On Thu, 2004-08-19 at 15:30, Andreas Tobler wrote:
> Hi,
>
> my darwin box is complaining :)
Weird that mine didn't. My apologies.
>
> /Volumes/src/gcc/gcc-cvs-clean/gcc/gcc/cp/tree.c:2136: warning:
> 'mark_local_for_remap_r' defined but not used
>
> Fixed with the appended patch, ok?
>
I think this qualifies as obvious.
-eric
> Regards,
> Andreas
>
> 2004-08-20 Andreas Tobler <a.tobler@schweiz.ch>
>
> * tree.c: Remove unused mark_local_for_remap_r.
>
> ______________________________________________________________________
> Index: gcc/cp/tree.c
> ===================================================================
> RCS file: /cvs/gcc/gcc/gcc/cp/tree.c,v
> retrieving revision 1.403
> diff -u -r1.403 tree.c
> --- gcc/cp/tree.c 19 Aug 2004 21:34:37 -0000 1.403
> +++ gcc/cp/tree.c 19 Aug 2004 22:26:52 -0000
> @@ -42,7 +42,6 @@
> static hashval_t list_hash_pieces (tree, tree, tree);
> static hashval_t list_hash (const void *);
> static cp_lvalue_kind lvalue_p_1 (tree, int);
> -static tree mark_local_for_remap_r (tree *, int *, void *);
> static tree build_target_expr (tree, tree);
> static tree count_trees_r (tree *, int *, void *);
> static tree verify_stmt_tree_r (tree *, int *, void *);
> @@ -2125,51 +2124,6 @@
> list_hash_table = htab_create_ggc (31, list_hash, list_hash_eq, NULL);
> }
>
> -/* Called via walk_tree. If *TP points to a DECL_EXPR for a local
> - declaration, copies the declaration and enters it in the splay_tree
> - pointed to by DATA (which is really a `splay_tree *'). */
> -
> -static tree
> -mark_local_for_remap_r (tree* tp,
> - int* walk_subtrees ATTRIBUTE_UNUSED ,
> - void* data)
> -{
> - tree t = *tp;
> - splay_tree st = (splay_tree) data;
> - tree decl;
> -
> -
> - if (TREE_CODE (t) == DECL_EXPR
> - && nonstatic_local_decl_p (DECL_EXPR_DECL (t)))
> - decl = DECL_EXPR_DECL (t);
> - else if (TREE_CODE (t) == LABEL_EXPR)
> - decl = LABEL_EXPR_LABEL (t);
> - else if (TREE_CODE (t) == TARGET_EXPR
> - && nonstatic_local_decl_p (TREE_OPERAND (t, 0)))
> - decl = TREE_OPERAND (t, 0);
> - else if (TREE_CODE (t) == CASE_LABEL_EXPR)
> - decl = CASE_LABEL (t);
> - else
> - decl = NULL_TREE;
> -
> - if (decl)
> - {
> - tree copy;
> -
> - /* Make a copy. */
> - copy = copy_decl_for_inlining (decl,
> - DECL_CONTEXT (decl),
> - DECL_CONTEXT (decl));
> -
> - /* Remember the copy. */
> - splay_tree_insert (st,
> - (splay_tree_key) decl,
> - (splay_tree_value) copy);
> - }
> -
> - return NULL_TREE;
> -}
> -
> /* Returns the kind of special function that DECL (a FUNCTION_DECL)
> is. Note that sfk_none is zero, so this function can be used as a
> predicate to test whether or not DECL is a special function. */
--
Eric Christopher <echristo@redhat.com>