[patch] remove unsave_expr langhook

Richard Henderson rth@redhat.com
Thu Aug 19 20:44:00 GMT 2004


On Thu, Aug 19, 2004 at 12:35:17PM -0700, Eric Christopher wrote:
> 	* langhooks-def.h (LANG_HOOKS_UNSAVE_EXPR_NOW): Remove.
> 	* langhooks.h (unsave_expr_now): Ditto.
> 	* tree.h (unsave_expr_1): Remove prototype.
> 	(lhd_unsave_expr_now): Rename to unsave_expr_now.
> 	* tree-inline.c (unsave_expr_1): Move here
> 	from tree.c. Make static.
> 	(unsave_expr_now): Rename from lhd_unsave_expr_now.
> 	* tree-sra.c: Fix up for rename.
> 	* tree-ssa-copy.c: Ditto.
> 	* tree-eh.c: Ditto.
> 	* tree.c (unsave_expr_1): Move to tree-inline.c.
> 
> 2004-08-19  Eric Christopher  <echristo@redhat.com>
> 
> 	* cp-tree.h (cxx_unsave_expr_now): Delete prototype.
> 	* tree.c (cxx_unsave_expr_now): Delete.
> 	(cp_unsave_r): Ditto.

Ok.

I wish we could come up with a better name than "unsave_expr_now" for
the remaining function, but we already have the obvious copy-related
names taken.  I guess we need to survey them and characterize how they
differ and either remove some, or rename them such that it's obvious
what characteristics we're after.


r~



More information about the Gcc-patches mailing list