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: PREFERRED_STACK_BOUNDARY/function calling code fix


>>>>> "Jan" == Jan Hubicka <hubicka@atrey.karlin.mff.cuni.cz> writes:

    Jan> Graham, it would be also nice idea to add your testcase to
    Jan> the testsuite.  I've bootstraped with older version of this
    Jan> patch, so before installing it I only run the testsuite, so
    Jan> it looks like we don't have any testcase for the nested
    Jan> functions calls, that was all broken after my patch.

I think it's fine to add things to the testsuite, even sometimes when
they're slightly redundant with the bootstrap.

But, we should always bootstrap.  I, myself, am sometimes sorely
tempted to make just one little tweak before checkin.  Although I
sometimes fail to do so, I try to will myself to resist that
temptation, unless its a change entirely contained within a comment.

--
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]