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