[RFA] reverse debugging tests
Tom Tromey
tromey@redhat.com
Fri Sep 11 22:29:00 GMT 2009
>>>>> "Michael" == Michael Snyder <msnyder@vmware.com> writes:
Michael> Note, I've only done one of the several tests here --
Michael> if this change is approved, I'll change the others the same way.
Thanks for doing this.
Michael> So I've made two changes:
Michael> 1) In place of "if ![target_info exists gdb,can_reverse]"
Michael> substitute
Michael> +if [istarget "*linux*"] then {
Michael> + if { ![istarget "i?86-*linux*"] && ![istarget "amd64-*linux*"] } then {
Michael> + # test process record on i?86 and amd64 linux
Michael> + return;
Michael> + }
Michael> +}
I assume the old gdb,can_reverse and gdb,use_precord target settings no
longer make sense.
I would recommend putting each of these snippets into a function in
lib/gdb.exp. That way it is simpler to update them in the future.
Michael> + gdb_test "record" "" "Turn on process record"
Michael> + # FIXME: command ought to acknowledge, so we can test if it
Michael> succeeded.
This seems like a good idea.
Otherwise this looks fine to me.
Tom
More information about the Gdb-patches
mailing list