Set breakpoint condition-evaluation in forking-threads-plus-breakpoint.exp
Commit Message
Proc do_test in forking-threads-plus-breakpoint.exp has an argument
cond_bp_target, but the test doesn't use it to set
"breakpoint condition-evaluation", which is an oversight in the test.
This patch fixes it by setting "breakpoint condition-evaluation" per
$cond_bp_target.
gdb/testsuite:
2016-02-18 Yao Qi <yao.qi@linaro.org>
* gdb.threads/forking-threads-plus-breakpoint.exp (do_test):
Set "set breakpoint condition-evaluation" per $cond_bp_target.
---
.../gdb.threads/forking-threads-plus-breakpoint.exp | 15 +++++++++++++++
1 file changed, 15 insertions(+)
Comments
On 02/18/2016 10:59 AM, Yao Qi wrote:
> Proc do_test in forking-threads-plus-breakpoint.exp has an argument
> cond_bp_target, but the test doesn't use it to set
> "breakpoint condition-evaluation", which is an oversight in the test.
Whoops, indeed.
>
> This patch fixes it by setting "breakpoint condition-evaluation" per
> $cond_bp_target.
>
> gdb/testsuite:
>
> 2016-02-18 Yao Qi <yao.qi@linaro.org>
>
> * gdb.threads/forking-threads-plus-breakpoint.exp (do_test):
> Set "set breakpoint condition-evaluation" per $cond_bp_target.
Thanks, LGTM.
Thanks,
Pedro Alves
Pedro Alves <palves@redhat.com> writes:
> Thanks, LGTM.
Patch is pushed in.
@@ -83,6 +83,21 @@ proc do_test { cond_bp_target detach_on_fork displaced } {
return 0
}
+ if {$cond_bp_target} {
+ set test "set breakpoint condition-evaluation target"
+ gdb_test_multiple $test $test {
+ -re "warning: Target does not support breakpoint condition evaluation.\r\nUsing host evaluation mode instead.\r\n$gdb_prompt $" {
+ # Target doesn't support breakpoint condition
+ # evaluation on its side. Skip the test.
+ return 0
+ }
+ -re "^$test\r\n$gdb_prompt $" {
+ }
+ }
+ } else {
+ gdb_test_no_output "set breakpoint condition-evaluation host"
+ }
+
gdb_test_no_output "set detach-on-fork $detach_on_fork"
gdb_test_no_output "set displaced $displaced"