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]

Use n_popped uniformly in emit_call_1()


Most of emit_call_1 use n_popped instead of RETURN_POPS_ARGS().  Only
the block the tests for sibcall_pop and sibcall_value_pop doesn't.
Probably an oversight in some merge.  I'm tempted to install this as
an obvious bug-fix but I'll resist and ask: ok to install? :-)

Index: gcc/ChangeLog
from  Alexandre Oliva  <aoliva@redhat.com>

	* calls.c (emit_call_1) [sibcall_pop]: Use n_popped instead of
	RETURN_POPS_ARGS().

Index: gcc/calls.c
--- gcc/calls.c	Fri Oct 20 07:03:00 2000
+++ gcc/calls.c	Tue Oct 31 06:34:39 2000
@@ -473,10 +473,9 @@ emit_call_1 (funexp, fndecl, funtype, st
 #if defined (HAVE_sibcall_pop) && defined (HAVE_sibcall_value_pop)
   if ((ecf_flags & ECF_SIBCALL)
       && HAVE_sibcall_pop && HAVE_sibcall_value_pop
-      && (RETURN_POPS_ARGS (fndecl, funtype, stack_size) > 0
-	  || stack_size == 0))
+      && (n_popped > 0 || stack_size == 0))
     {
-      rtx n_pop = GEN_INT (RETURN_POPS_ARGS (fndecl, funtype, stack_size));
+      rtx n_pop = GEN_INT (n_popped));
       rtx pat;
 
       /* If this subroutine pops its own args, record that in the call insn

-- 
Alexandre Oliva   Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer                  aoliva@{cygnus.com, redhat.com}
CS PhD student at IC-Unicamp        oliva@{lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist    *Please* write to mailing lists, not to me

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