This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Simulator testing for sh and sh64


On Thu, 23 Feb 2012 20:49:50 +0100
Thomas Schwinge <thomas@codesourcery.com> wrote:

> Anyway, the patch for sh-tdep that I posted in
> <http://sourceware.org/ml/gdb-patches/2012-02/msg00299.html> (at the end)
> also applies to sh64-tdep -- shall I commit the equivalent sh64-tdep
> change without any testsuite testing, or let it bit-rot some more?

I would like you to apply that patch to sh-tdep.c.  (So consider
that patch approved for sh-tdep.c.)

I'd also like to see some version of my patch go in if it makes
sense.  But I'd like to see your patch applied as a first step.
(I sent a similar reply on Monday.)

With regard to sh64-tdep.c...  Are you able to do any testing at all
to make sure that your patch basically works for sh64?  If you're
able to get partial results with the other changes that you've made,
I think that's good enough.  Even hand testing on something like
the gdb.base/break.exp test case would be okay.

So... if you're able to test it at all so that you know it basically
works, then it can go in.  If not, I'd prefer to have sh64-tdep.c left
in its current state until you are able to do some testing.

Kevin


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