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]

Re: C++ PATCH: Remove THUNK_DECLs


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

    Jason> I don't think this is safe.  The reason for using a
    Jason> THUNK_DECL in the first place was because we wanted to
    Jason> store information in it that is wrong for a FUNCTION_DECL:

    Jason> DECL_INITIAL points to a FUNCTION_DECL, rather than a
    Jason> BLOCK.  THUNK_DELTA <=> DECL_FRAME_SIZE THUNK_VCALL_OFFSET
    Jason> <=> DECL_SAVED_INSNS

I've just looked a little more closely.  Now I'm a little more
nervous.

    Jason> The last sounds particularly dangerous.  I would not be
    Jason> opposed to moving this information into DECL_LANG_SPECIFIC,
    Jason> though.

That's probably a good suggestion.

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

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