Re: [PATCH] arm reversible : <phase_2_complete>

Yao Qi
2011-10-15T13:33:43Z

On 10/15/2011 05:31 PM, Yao Qi wrote:
> I noticed that test cases in gdb.reverse are *not* enabled in default.
> You need to add the following two lines in your board file to turn them on,
>
> set_board_info gdb,use_precord 1
> set_board_info gdb,can_reverse 1
>
> It is good if you can post the test result in gdb.reverse so that we are
> more confidient to your patch.
^^^^^^^^^^ typo "confident".

I run test cases in gdb.reverse with my own board file [1], and get
following result on x86:

=== gdb Summary ===

# of expected passes 2774
# of unexpected failures 22

I also applied your patch, and run gdb.reverse test cases on arm natively:

=== gdb Summary ===

# of expected passes 1180
# of unexpected failures 1090
# of expected failures 142

I don't think these fails are all related to your arm-reverse patch.
IMO, it means there are still some work to do, not only in arm bits, but
also in gdb general parts and test cases. Sorry that I can't give any
useful suggestion here.

--
Yao (é½å°§)

[1] My board file reverse.exp

load_generic_config "unix"
process_multilib_options ""

# The default compiler for this target.
set_board_info compiler "[find_gcc]"

set_board_info gdb,can_reverse 1
set_board_info gdb,use_precord 1