C++ PATCH: Remove THUNK_DECLs

Mark Mitchell mark@codesourcery.com
Thu May 4 17:19:00 GMT 2000


>>>>> "Jason" == Jason Merrill <jason@cygnus.com> writes:

    THUNK_DELTA <=> DECL_FRAME_SIZE

DECL_FRAME_SIZE is actually almost totally unused in the current tree.
The C front-end copies it around, but never sets it.  The only read
from there is in integrate.c:

	  if (DECL_FRAME_SIZE (fndecl) != 0)
	    copy_rtx_and_substitute (virtual_stack_vars_rtx, map, 0);

But it's never set by the back-end.

It's used in gcj in FUNCTION_DECLs for an entirely different purpose
-- implying that this test is probably firing randomly for gcj.

It seems to me that this should just be removed.  That's how I intend
to solve that one of the issues you raised -- but DECL_SAVED_INSNS has
no such tidy fix.

Does anyone know of a reason DECL_FRAME_SIZE should stay?

(I won't actually make this change until I get back from vacation --
but I also won't worry about the possible overlap in the C++ front-end
until then either.)

--
Mark Mitchell                   mark@codesourcery.com
CodeSourcery, LLC               http://www.codesourcery.com


More information about the Gcc-patches mailing list