Re: [PATCH] arm reversible : <phase_2_complete>

[Yao Qi <yao at codesourcery dot com>]

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