This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[patch] Remove EXPAND_BUILTIN_VA_END.


Hi,

Attached is a patch to remove EXPAND_BUILTIN_VA_END as it is unused.

To be honest, EXPAND_BUILTIN_VA_{ARG|START} are defined in many ports,
so removing only EXPAND_BUILTIN_VA_END feels sort of strange.

Bootstrapped on i686-pc-linux-gnu.  OK to apply?

Kazu Hirata

2004-01-14  Kazu Hirata  <kazu@cs.umass.edu>

	* builtins.c (expand_builtin_va_end): Don't use
	EXPAND_BUILTIN_VA_END.
	* system.h (EXPAND_BUILTIN_VA_END): Poison.
	* config/d30v/d30v.h: Remove a commented-out definition of
	EXPAND_BUILTIN_VA_END.
	* config/stormy16/stormy16.h: Likewise.

Index: builtins.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/builtins.c,v
retrieving revision 1.273
diff -u -r1.273 builtins.c
--- builtins.c	13 Jan 2004 01:32:42 -0000	1.273
+++ builtins.c	14 Jan 2004 06:02:30 -0000
@@ -4143,15 +4143,10 @@
 {
   tree valist = TREE_VALUE (arglist);
 
-#ifdef EXPAND_BUILTIN_VA_END
-  valist = stabilize_va_list (valist, 0);
-  EXPAND_BUILTIN_VA_END (arglist);
-#else
   /* Evaluate for side effects, if needed.  I hate macros that don't
      do that.  */
   if (TREE_SIDE_EFFECTS (valist))
     expand_expr (valist, const0_rtx, VOIDmode, EXPAND_NORMAL);
-#endif
 
   return const0_rtx;
 }
Index: system.h
===================================================================
RCS file: /cvs/gcc/gcc/gcc/system.h,v
retrieving revision 1.184
diff -u -r1.184 system.h
--- system.h	14 Jan 2004 00:49:00 -0000	1.184
+++ system.h	14 Jan 2004 06:02:30 -0000
@@ -622,7 +622,8 @@
 	LIBGCC_NEEDS_DOUBLE FINAL_PRESCAN_LABEL DEFAULT_CALLER_SAVES	   \
 	LOAD_ARGS_REVERSED MAX_INTEGER_COMPUTATION_MODE			   \
 	CONVERT_HARD_REGISTER_TO_SSA_P ASM_OUTPUT_MAIN_SOURCE_FILENAME	   \
-	FIRST_INSN_ADDRESS TEXT_SECTION SHARED_BSS_SECTION_ASM_OP
+	FIRST_INSN_ADDRESS TEXT_SECTION SHARED_BSS_SECTION_ASM_OP	   \
+	EXPAND_BUILTIN_VA_END
 
 /* Hooks that are no longer used.  */
  #pragma GCC poison LANG_HOOKS_FUNCTION_MARK LANG_HOOKS_FUNCTION_FREE	\
Index: config/d30v/d30v.h
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/d30v/d30v.h,v
retrieving revision 1.87
diff -u -r1.87 d30v.h
--- config/d30v/d30v.h	30 Oct 2003 02:02:35 -0000	1.87
+++ config/d30v/d30v.h	14 Jan 2004 06:02:37 -0000
@@ -1421,13 +1421,6 @@
 
 #define EXPAND_BUILTIN_VA_ARG(VALIST, TYPE)				\
 (d30v_expand_builtin_va_arg (VALIST, TYPE))
-
-/* Implement the stdarg/varargs va_end macro.
-   VALIST is the variable of type va_list as a tree.  */
-
-/* #define EXPAND_BUILTIN_VA_END(VALIST) */
-
-
 
 /* Trampolines for Nested Functions.  */
 
Index: config/stormy16/stormy16.h
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/stormy16/stormy16.h,v
retrieving revision 1.81
diff -u -r1.81 stormy16.h
--- config/stormy16/stormy16.h	17 Nov 2003 08:42:59 -0000	1.81
+++ config/stormy16/stormy16.h	14 Jan 2004 06:02:40 -0000
@@ -508,11 +508,6 @@
    va_list as a tree, TYPE is the type passed to va_arg.  */
 #define EXPAND_BUILTIN_VA_ARG(VALIST, TYPE) \
   xstormy16_expand_builtin_va_arg (VALIST, TYPE)
-
-/* Implement the stdarg/varargs va_end macro.  VALIST is the variable of type
-   va_list as a tree.  */
-/* #define EXPAND_BUILTIN_VA_END(VALIST) */
-
 
 /* Trampolines for Nested Functions.  */
 


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]