From: Jim Blandy <jimb@zwingli.cygnus.com>
To: gdb-patches@sources.redhat.com
Subject: Re: RFA: recognize unavailable struct values in call-rt-st.exp
Date: Fri, 07 Dec 2001 13:24:00 -0000 [thread overview]
Message-ID: <npn10u67ti.fsf@zwingli.cygnus.com> (raw)
In-Reply-To: Jim Blandy's message of Fri, 30 Nov 2001 02:10:37 -0500 (EST)
I've committed this change.
Jim Blandy <jimb@cygnus.com> writes:
> More of the same.
>
> I've tested this on the S/390, which can't find structs returned by
> value, and the i386, which can. Both cases yielded the expected
> results: all unsupported, or all pass.
>
> 2001-11-30 Jim Blandy <jimb@redhat.com>
>
> If GDB says it can't find the struct the function returned, report
> those tests as `unsupported'.
> * gdb.base/call-rt-st.exp (print_struct_call): New function.
> Rewrite subsequent tests to use it.
>
> Index: gdb/testsuite/gdb.base/call-rt-st.exp
> ===================================================================
> RCS file: /cvs/cvsfiles/devo/gdb/testsuite/gdb.base/call-rt-st.exp,v
> retrieving revision 1.8
> diff -c -r1.8 call-rt-st.exp
> *** gdb/testsuite/gdb.base/call-rt-st.exp 2001/07/27 23:53:16 1.8
> --- gdb/testsuite/gdb.base/call-rt-st.exp 2001/11/30 07:03:20
> ***************
> *** 135,252 ****
> timeout { fail "(timeout)finish out from loop_count"}
> }
>
> ! if ![gdb_skip_stdio_test "print print_struct_rep(*struct1)"] {
> ! send_gdb "print print_struct_rep(*struct1)\n"
> ! gdb_expect {
> ! -re ".*Contents of struct1:\[ \t\n\r\]+22\[ \t\]+0\[ \t\n\r\]+.\[0-9\]+ = \\{value = 5, head = 0\\}.*$gdb_prompt $" {
> ! pass "print print_struct_rep(*struct1, *struct2, *struct3)"
> ! }
> ! -re ".*$gdb_prompt $" {
> ! fail "print print_struct_rep(*struct1, *struct2, *struct3)"
> ! }
> ! timeout {
> ! fail "(timeout) print_struct_rep(*struct1, *struct2, *struct3)"
> ! }
> }
> }
>
> if ![gdb_skip_stdio_test "print print_one_large_struct(...)"] {
> ! send_gdb "print print_one_large_struct(*list1)\n"
> ! gdb_expect {
> ! -re ".*\[ \t\]+4\[ \t\]+1\[ \r\n\]+.\[0-9\]+ = \\{next_index = \\{1, 2, 3, 4, 5, 6, 7, 8, 9, 10\\}, values = \\{4, 6, 8, 10, 12, 14, 16, 18, 20, 22\\}, head = 0\\}.*$gdb_prompt $" {
> ! pass "print print_one_large_struct(*list1)"
> ! }
> ! -re ".*$gdb_prompt $" { fail "print print_one_large_struct(*list1)" }
> ! timeout { fail "(timeout) print_one_large_struct(*list1)" }
> ! }
> }
>
> if {![gdb_skip_float_test "print print_one_double(*d1)"] && \
> ! ![gdb_skip_stdio_test "print print_one_double(*d1)"] } {
> ! send_gdb "print print_one_double(*d1)\n";
> ! gdb_expect {
> ! -re ".*Contents of one_double_t:\[ \r\n\]+1\\.111110\[ \r\n\]+.\[0-9\]+ = \\{double1 = 1\\.11111\\}.*$gdb_prompt $" {
> ! pass "print print_one_double(*d1)"
> ! }
> ! -re ".*$gdb_prompt $" { fail "print print_one_double(*d1)" }
> ! timeout { fail "(timeout) print_one_double(*d1)" }
> ! }
> }
>
> if {![gdb_skip_float_test "print print_two_floats(*f3)"] && \
> ! ![gdb_skip_stdio_test "print print_two_floats(*f3)"] } {
> ! send_gdb "print print_two_floats(*f3)\n"
> ! gdb_expect {
> ! -re ".*Contents of two_floats_t:\[ \r\n\]+-2\\.345000\[ \t]+1\\.000000\[ \r\n\]+.\[0-9\]+ = \\{float1 = -2\\.34500003, float2 = 1\\}.*$gdb_prompt $" {
> ! pass "print print_two_floats(*f3)"
> ! }
> ! -re ".*$gdb_prompt $" { fail "print print_two_floats(*f3)" }
> ! timeout { fail "(timeout) print_two_floats(*f3)" }
> ! }
> }
>
> if ![gdb_skip_stdio_test "print print_bit_flags(*flags)"] {
> ! send_gdb "print print_bit_flags(*flags)\n"
> ! gdb_expect {
> ! -re ".*alpha\[ \r\n\]+gamma\[ \r\n\]+epsilon\[ \r\n\]+.\[0-9\]+ = \\{alpha = 1, beta = 0, gamma = 1, delta = 0, epsilon = 1, omega = 0\\}\[ \r\n\]+$gdb_prompt $" {
> ! pass "print print_bit_flags(*flags)"
> ! }
> ! -re ".*$gdb_prompt $" { fail "print print_bit_flags(*flags)" }
> ! timeout { fail "(timeout) print_bit_flags(*flags)" }
> ! }
> }
>
> if ![gdb_skip_stdio_test "print print_bit_flags_combo(*flags_combo)"] {
> ! send_gdb "print print_bit_flags_combo(*flags_combo)\n"
> ! gdb_expect {
> ! -re ".*alpha\[ \r\n\]+gamma\[ \r\n\]+epsilon\[ \r\n\]+ch1: y\[ \t\]+ch2: n\[ \r\n\]+.\[0-9\]+ = \\{alpha = 1, beta = 0, ch1 = 121 'y', gamma = 1, delta = 0, ch2 = 110 'n', epsilon = 1, omega = 0\\}\[ \r\n\]+$gdb_prompt $" {
> ! pass "print print_bit_flags_combo(*flags_combo)"
> ! }
> ! -re ".*$gdb_prompt $" {
> ! fail "print print_bit_flags_combo(*flags_combo)"
> ! }
> ! timeout {
> ! fail "(timeout) print_bit_flags_combo(*flags_combo)"
> ! }
> ! }
> }
>
> if ![gdb_skip_stdio_test "print print_three_chars(*three_chars)"] {
> ! send_gdb "print print_three_chars(*three_char)\n"
> ! gdb_expect {
> ! -re ".*Contents of three_char_t:\[ \r\n\]+x\[ \t\]+y\[ \t\]+z\[ \r\n\]+.\[0-9\]+ = \\{ch1 = 120 'x', ch2 = 121 'y', ch3 = 122 'z'\\}\[ \r\n\]+$gdb_prompt $" {
> ! pass "print print_three_chars(*three_char)"
> ! }
> ! -re ".*$gdb_prompt $" { fail "print print_three_chars(*three_char)" }
> ! timeout { fail "(timeout) print_three_chars(*three_char)" }
> ! }
> }
>
> if ![gdb_skip_stdio_test "print print_five_chars(*five_chars)"] {
> ! send_gdb "print print_five_chars(*five_char)\n"
> ! gdb_expect {
> ! -re ".*Contents of five_char_t:\[ \r\n\]+h\[ \t\]+e\[ \t\]+l\[ \t\]+l\[ \t\]+o\[ \r\n\]+.\[0-9\]+ = \\{ch1 = 104 'h', ch2 = 101 'e', ch3 = 108 'l', ch4 = 108 'l', ch5 = 111 'o'\\}\[ \r\n\]+$gdb_prompt $" {
> ! pass "print print_five_chars(*five_char)"
> ! }
> ! -re ".*$gdb_prompt $" { fail "print print_five_chars(*five_char)" }
> ! timeout { fail "(timeout) print_five_chars(*five_char)" }
> ! }
> }
>
> if ![gdb_skip_stdio_test "print print_int_char_combo(*int_char_combo)"] {
> ! send_gdb "print print_int_char_combo(*int_char_combo)\n"
> ! gdb_expect {
> ! -re ".*Contents of int_char_combo_t:\[ \r\n\]+13\[ \t\]+!\[ \r\n\]+.\[0-9\]+ = \\{int1 = 13, ch1 = 33 '!'\\}\[ \r\n\]+$gdb_prompt $" {
> ! pass "print print_int_char_combo(*int_char_combo)"
> ! }
> ! -re ".*$gdb_prompt $" {
> ! fail "print print_int_char_combo(*int_char_combo)"
> ! }
> ! timeout {
> ! fail "(timeout) print_int_char_combo(*int_char_combo)"
> ! }
> ! }
> }
>
> return
> -
> --- 135,212 ----
> timeout { fail "(timeout)finish out from loop_count"}
> }
>
> ! # Ask GDB to print the value of EXPR, and expect to see the regexp
> ! # RESULT in the output. If we get back the error message "Function
> ! # return value unknown", call that an `unsupported' test; on some
> ! # architectures, it's impossible to find structs returned by value
> ! # reliably.
> ! proc print_struct_call { expr result } {
> ! global gdb_prompt
> !
> ! set command "print $expr"
> ! send_gdb "${command}\n"
> ! gdb_expect {
> ! -re "$result\[\r\n\]+$gdb_prompt $" {
> ! pass "$command"
> ! }
> ! -re "Function return value unknown.\[\r\n\]+$gdb_prompt $" {
> ! unsupported "$command"
> ! }
> ! -re "$gdb_prompt $" {
> ! fail "$command"
> ! }
> ! timeout {
> ! fail "$command (timeout)"
> ! }
> }
> }
>
> +
> + if ![gdb_skip_stdio_test "print print_struct_rep(*struct1)"] {
> + print_struct_call "print_struct_rep(*struct1)" \
> + ".*Contents of struct1:\[ \t\n\r\]+22\[ \t\]+0\[ \t\n\r\]+.\[0-9\]+ = \\{value = 5, head = 0\\}"
> + }
> +
> if ![gdb_skip_stdio_test "print print_one_large_struct(...)"] {
> ! print_struct_call "print_one_large_struct(*list1)" \
> ! ".*\[ \t\]+4\[ \t\]+1\[ \r\n\]+.\[0-9\]+ = \\{next_index = \\{1, 2, 3, 4, 5, 6, 7, 8, 9, 10\\}, values = \\{4, 6, 8, 10, 12, 14, 16, 18, 20, 22\\}, head = 0\\}"
> }
>
> if {![gdb_skip_float_test "print print_one_double(*d1)"] && \
> ! ![gdb_skip_stdio_test "print print_one_double(*d1)"] } {
> ! print_struct_call "print_one_double(*d1)" \
> ! ".*Contents of one_double_t:\[ \r\n\]+1\\.111110\[ \r\n\]+.\[0-9\]+ = \\{double1 = 1\\.11111\\}"
> }
>
> if {![gdb_skip_float_test "print print_two_floats(*f3)"] && \
> ! ![gdb_skip_stdio_test "print print_two_floats(*f3)"] } {
> ! print_struct_call "print_two_floats(*f3)" \
> ! ".*Contents of two_floats_t:\[ \r\n\]+-2\\.345000\[ \t]+1\\.000000\[ \r\n\]+.\[0-9\]+ = \\{float1 = -2\\.34500003, float2 = 1\\}"
> }
>
> if ![gdb_skip_stdio_test "print print_bit_flags(*flags)"] {
> ! print_struct_call "print_bit_flags(*flags)" \
> ! ".*alpha\[ \r\n\]+gamma\[ \r\n\]+epsilon\[ \r\n\]+.\[0-9\]+ = \\{alpha = 1, beta = 0, gamma = 1, delta = 0, epsilon = 1, omega = 0\\}"
> }
>
> if ![gdb_skip_stdio_test "print print_bit_flags_combo(*flags_combo)"] {
> ! print_struct_call "print_bit_flags_combo(*flags_combo)" \
> ! ".*alpha\[ \r\n\]+gamma\[ \r\n\]+epsilon\[ \r\n\]+ch1: y\[ \t\]+ch2: n\[ \r\n\]+.\[0-9\]+ = \\{alpha = 1, beta = 0, ch1 = 121 'y', gamma = 1, delta = 0, ch2 = 110 'n', epsilon = 1, omega = 0\\}"
> }
>
> if ![gdb_skip_stdio_test "print print_three_chars(*three_chars)"] {
> ! print_struct_call "print_three_chars(*three_char)" \
> ! ".*Contents of three_char_t:\[ \r\n\]+x\[ \t\]+y\[ \t\]+z\[ \r\n\]+.\[0-9\]+ = \\{ch1 = 120 'x', ch2 = 121 'y', ch3 = 122 'z'\\}"
> }
>
> if ![gdb_skip_stdio_test "print print_five_chars(*five_chars)"] {
> ! print_struct_call "print_five_chars(*five_char)" \
> ! ".*Contents of five_char_t:\[ \r\n\]+h\[ \t\]+e\[ \t\]+l\[ \t\]+l\[ \t\]+o\[ \r\n\]+.\[0-9\]+ = \\{ch1 = 104 'h', ch2 = 101 'e', ch3 = 108 'l', ch4 = 108 'l', ch5 = 111 'o'\\}"
> }
>
> if ![gdb_skip_stdio_test "print print_int_char_combo(*int_char_combo)"] {
> ! print_struct_call "print_int_char_combo(*int_char_combo)" \
> ! ".*Contents of int_char_combo_t:\[ \r\n\]+13\[ \t\]+!\[ \r\n\]+.\[0-9\]+ = \\{int1 = 13, ch1 = 33 '!'\\}"
> }
>
> return
>
prev parent reply other threads:[~2001-12-07 21:24 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-11-29 23:09 Jim Blandy
2001-11-21 13:15 ` Jim Blandy
2001-12-07 13:24 ` Jim Blandy [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=npn10u67ti.fsf@zwingli.cygnus.com \
--to=jimb@zwingli.cygnus.com \
--cc=gdb-patches@sources.redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox