espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-06-20 09:58:19 +08:00
Code Issues Packages Projects Releases Wiki Activity

gdb/testsuite: make gdb_breakpoint and runto take a linespec

Browse Source 
Change gdb_breakpoint to accept a linespec, not just a function.  In
fact, no behavior changes are necessary, this only changes the parameter
name and documentation.  Change runto as well, since the two are so
close (runto forwards all its arguments to gdb_breakpoint).

I wrote this for a downstrean GDB port,  but thought it could be
useful upstream, eventually, even though not callers take advantage of
it yet.

Change-Id: I08175fd444d5a60df90fd9985e1b5dfd87c027cc
This commit is contained in:
Simon Marchi
2022-02-08 16:31:09 -05:00
parent 524ad5e30f
commit d3fc98f911
1 changed files with 13 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
gdb/testsuite/lib/gdb.exp
View File

@ -546,9 +546,11 @@ proc gdb_starti_cmd { {inferior_args {}} } {
return -1 return -1
} }
# Set a breakpoint at FUNCTION. If there is an additional argument it is # Set a breakpoint using LINESPEC.
# a list of options; the supported options are allow-pending, temporary, #
# message, no-message and qualified. # If there is an additional argument it is a list of options; the supported
# options are allow-pending, temporary, message, no-message and qualified.
#
# The result is 1 for success, 0 for failure. # The result is 1 for success, 0 for failure.
# #
# Note: The handling of message vs no-message is messed up, but it's based # Note: The handling of message vs no-message is messed up, but it's based
@ -557,7 +559,7 @@ proc gdb_starti_cmd { {inferior_args {}} } {
# no-message: turns off printing of fails (and passes, but they're already off) # no-message: turns off printing of fails (and passes, but they're already off)
# message: turns on printing of passes (and fails, but they're already on) # message: turns on printing of passes (and fails, but they're already on)
proc gdb_breakpoint { function args } { proc gdb_breakpoint { linespec args } {
global gdb_prompt global gdb_prompt
global decimal global decimal
@ -588,9 +590,9 @@ proc gdb_breakpoint { function args } {
set print_pass 1 set print_pass 1
} }
set test_name "setting breakpoint at $function" set test_name "gdb_breakpoint: set breakpoint at $linespec"
send_gdb "$break_command $function\n" send_gdb "$break_command $linespec\n"
# The first two regexps are what we get with -g, the third is without -g. # The first two regexps are what we get with -g, the third is without -g.
gdb_expect 30 { gdb_expect 30 {
-re "$break_message \[0-9\]* at .*: file .*, line $decimal.\r\n$gdb_prompt $" {} -re "$break_message \[0-9\]* at .*: file .*, line $decimal.\r\n$gdb_prompt $" {}
@ -658,7 +660,7 @@ proc gdb_breakpoint { function args } {
# no-message: turns off printing of fails (and passes, but they're already off) # no-message: turns off printing of fails (and passes, but they're already off)
# message: turns on printing of passes (and fails, but they're already on) # message: turns on printing of passes (and fails, but they're already on)
proc runto { function args } { proc runto { linespec args } {
global gdb_prompt global gdb_prompt
global decimal global decimal
@ -675,14 +677,14 @@ proc runto { function args } {
set print_pass 1 set print_pass 1
} }
set test_name "running to $function in runto" set test_name "runto: run to $linespec"
# We need to use eval here to pass our varargs args to gdb_breakpoint # We need to use eval here to pass our varargs args to gdb_breakpoint
# which is also a varargs function. # which is also a varargs function.
# But we also have to be careful because $function may have multiple # But we also have to be careful because $linespec may have multiple
# elements, and we don't want Tcl to move the remaining elements after # elements, and we don't want Tcl to move the remaining elements after
# the first to $args. That is why $function is wrapped in {}. # the first to $args. That is why $linespec is wrapped in {}.
if ![eval gdb_breakpoint {$function} $args] { if ![eval gdb_breakpoint {$linespec} $args] {
return 0 return 0
} }