Re: [PATCH] Add testcases to gdb.base/break.exp

[Andrew Burgess <andrew.burgess@embecosm.com>]

* Fei Jie <feij.fnst@cn.fujitsu.com> [2015-10-27 13:38:57 +0800]:

>     *test break at offset -1
>     *test break with address
>     *test break with non-existed address
>     *test break with non-existed condition
> ---
>  gdb/testsuite/gdb.base/break.exp | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
> 
> diff --git a/gdb/testsuite/gdb.base/break.exp b/gdb/testsuite/gdb.base/break.exp
> index f879bc8..3a997d1 100644
> --- a/gdb/testsuite/gdb.base/break.exp
> +++ b/gdb/testsuite/gdb.base/break.exp
> @@ -399,6 +399,10 @@ set bp_location10 [gdb_get_line_number "set breakpoint 10 here"]
>  gdb_test "break +1" \
>      "Breakpoint.*at.* file .*$srcfile, line $bp_location10\\." \
>      "breakpoint offset +1"
> +#Test break at offset -1
> +gdb_test "break -1" \
> +    "Note: breakpoint 6 also set at.*Breakpoint.*at.* file .*$srcfile, line.*" \
> +    "breakpoint offset -1"
>  
>  # Check to see if breakpoint is hit when stepped onto
>  
> @@ -940,3 +944,17 @@ gdb_test_no_output "set \$foo=81.5" \
>  gdb_test "break $srcfile:\$foo" \
>      "Convenience variables used in line specs must have integer values.*" \
>      "set breakpoint via non-integer convenience variable disallowed"
> +
> +#Test break with address
> +gdb_test "break *0x400795" \
> +    "Breakpoint.*at.* .*$srcfile, line.*" \
> +    "breakpoint with address"

That address is not going to be valid for every target.  You'll need
to place a label in the code somewhere, take the address of the label,
and use that to generate a breakpoint.

Thanks,
Andrew