* [pushed 0/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host/target
@ 2023-03-22 8:38 Tom de Vries
2023-03-22 8:38 ` [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host Tom de Vries
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Tom de Vries @ 2023-03-22 8:38 UTC (permalink / raw)
To: gdb-patches
Fix a few test-cases in gdb.cp/*.exp for various remote host/target boards.
Results:
...
$ ./test-all.sh
HOST/TARGET: local
# of expected failures 14
# of expected passes 8576
# of known failures 21
HOST: local, TARGET: native-gdbserver
# of expected failures 14
# of expected passes 8552
# of known failures 21
# of unsupported tests 2
HOST: local, TARGET: native-extended-gdbserver
# of expected failures 14
# of expected passes 8576
# of known failures 21
HOST: local, TARGET: native-stdio-gdbserver
# of expected failures 14
# of expected passes 8552
# of known failures 21
# of unsupported tests 2
HOST: local, TARGET: remote-gdbserver-on-localhost
# of expected failures 14
# of expected passes 8560
# of known failures 21
# of unsupported tests 2
HOST: local, TARGET: remote-stdio-gdbserver
# of expected failures 14
# of expected passes 8560
# of known failures 21
# of unsupported tests 2
HOST: local, TARGET: remote-stdio-gdbserver-remotedir
# of expected failures 14
# of expected passes 8560
# of known failures 21
# of unsupported tests 2
HOST: local-remote-host-notty, TARGET: native-gdbserver
# of expected failures 14
# of expected passes 8540
# of known failures 21
# of unsupported tests 2
# of untested testcases 1
HOST: local-remote-host-notty, TARGET: native-extended-gdbserver
# of expected failures 14
# of expected passes 8564
# of known failures 21
# of untested testcases 1
HOST: local-remote-host-notty, TARGET: remote-gdbserver-on-localhost
# of expected failures 14
# of expected passes 8540
# of known failures 21
# of unsupported tests 2
# of untested testcases 1
HOST: local-remote-host-notty, TARGET: remote-stdio-gdbserver
# of expected failures 14
# of expected passes 8540
# of known failures 21
# of unsupported tests 2
# of untested testcases 1
HOST: local-remote-host-notty, TARGET: remote-stdio-gdbserver-remotedir
# of expected failures 14
# of expected passes 8540
# of known failures 21
# of unsupported tests 2
# of untested testcases 1
HOST: local-remote-host, TARGET: native-gdbserver
# of expected failures 14
# of expected passes 8560
# of known failures 21
# of unsupported tests 2
HOST: local-remote-host, TARGET: native-extended-gdbserver
# of expected failures 14
# of expected passes 8584
# of known failures 21
HOST: local-remote-host, TARGET: remote-gdbserver-on-localhost
# of expected failures 14
# of expected passes 8560
# of known failures 21
# of unsupported tests 2
HOST: local-remote-host, TARGET: remote-stdio-gdbserver
# of expected failures 14
# of expected passes 8560
# of known failures 21
# of unsupported tests 2
HOST: local-remote-host, TARGET: remote-stdio-gdbserver-remotedir
# of expected failures 14
# of expected passes 8560
# of known failures 21
# of unsupported tests 2
HOST/TARGET: local-remote-host-native
# of expected failures 14
# of expected passes 8584
# of known failures 21
...
Tom de Vries (3):
[gdb/testsuite] Fix gdb.cp/*.exp for remote host
[gdb/testsuite] Fix gdb.cp/annota{2,3}.exp for
native-extended-gdbserver
[gdb/testsuite] Fix gdb.cp/cp-relocate.exp for remote host
gdb/testsuite/gdb.cp/annota2.exp | 15 ++++++++++++--
gdb/testsuite/gdb.cp/annota3.exp | 25 ++++++++++++++++--------
gdb/testsuite/gdb.cp/breakpoint-locs.exp | 2 ++
gdb/testsuite/gdb.cp/cp-relocate.exp | 4 +++-
gdb/testsuite/gdb.cp/filename.exp | 2 ++
gdb/testsuite/gdb.cp/gdb2384.exp | 2 ++
gdb/testsuite/gdb.cp/hang.exp | 2 ++
gdb/testsuite/gdb.cp/m-static.exp | 8 +++++++-
gdb/testsuite/gdb.cp/mb-inline.exp | 1 +
gdb/testsuite/gdb.cp/minsym-fallback.exp | 3 +++
gdb/testsuite/gdb.cp/ovsrch.exp | 2 ++
gdb/testsuite/gdb.cp/pr10728.exp | 2 ++
gdb/testsuite/gdb.cp/rtti.exp | 2 ++
gdb/testsuite/gdb.cp/stub-array-size.exp | 2 ++
gdb/testsuite/gdb.cp/subtypes.exp | 2 ++
gdb/testsuite/lib/gdb.exp | 12 ++++++++++++
16 files changed, 74 insertions(+), 12 deletions(-)
base-commit: 100f993c53a50352fd91554927142a2558f3481c
--
2.35.3
^ permalink raw reply [flat|nested] 7+ messages in thread
* [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host
2023-03-22 8:38 [pushed 0/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host/target Tom de Vries
@ 2023-03-22 8:38 ` Tom de Vries
2023-03-22 15:05 ` Simon Marchi
2023-03-22 8:38 ` [pushed 2/3] [gdb/testsuite] Fix gdb.cp/annota{2,3}.exp for native-extended-gdbserver Tom de Vries
2023-03-22 8:38 ` [pushed 3/3] [gdb/testsuite] Fix gdb.cp/cp-relocate.exp for remote host Tom de Vries
2 siblings, 1 reply; 7+ messages in thread
From: Tom de Vries @ 2023-03-22 8:38 UTC (permalink / raw)
To: gdb-patches
Fix a few test-cases in gdb.cp/*.exp for remote host using new proc
include_file.
Tested on x86_64-linux.
---
gdb/testsuite/gdb.cp/breakpoint-locs.exp | 2 ++
gdb/testsuite/gdb.cp/filename.exp | 2 ++
gdb/testsuite/gdb.cp/gdb2384.exp | 2 ++
gdb/testsuite/gdb.cp/hang.exp | 2 ++
gdb/testsuite/gdb.cp/m-static.exp | 8 +++++++-
gdb/testsuite/gdb.cp/mb-inline.exp | 1 +
gdb/testsuite/gdb.cp/minsym-fallback.exp | 3 +++
gdb/testsuite/gdb.cp/ovsrch.exp | 2 ++
gdb/testsuite/gdb.cp/pr10728.exp | 2 ++
gdb/testsuite/gdb.cp/rtti.exp | 2 ++
gdb/testsuite/gdb.cp/stub-array-size.exp | 2 ++
gdb/testsuite/gdb.cp/subtypes.exp | 2 ++
gdb/testsuite/lib/gdb.exp | 12 ++++++++++++
13 files changed, 41 insertions(+), 1 deletion(-)
diff --git a/gdb/testsuite/gdb.cp/breakpoint-locs.exp b/gdb/testsuite/gdb.cp/breakpoint-locs.exp
index 0d1ad68f9f5..11192654a35 100644
--- a/gdb/testsuite/gdb.cp/breakpoint-locs.exp
+++ b/gdb/testsuite/gdb.cp/breakpoint-locs.exp
@@ -19,6 +19,8 @@ require allow_cplus_tests
standard_testfile .cc breakpoint-locs-2.cc
+include_file breakpoint-locs.h
+
if { [prepare_for_testing "failed to prepare" $testfile "$srcfile $srcfile2"\
{debug c++}] } {
return -1
diff --git a/gdb/testsuite/gdb.cp/filename.exp b/gdb/testsuite/gdb.cp/filename.exp
index 9b1d2b8a606..2eddb9b7982 100644
--- a/gdb/testsuite/gdb.cp/filename.exp
+++ b/gdb/testsuite/gdb.cp/filename.exp
@@ -17,6 +17,8 @@ require allow_cplus_tests
standard_testfile .cc
+include_file includefile
+
if {[prepare_for_testing "failed to prepare" $testfile $srcfile {debug c++}]} {
return -1
}
diff --git a/gdb/testsuite/gdb.cp/gdb2384.exp b/gdb/testsuite/gdb.cp/gdb2384.exp
index 46d93d27e7b..7c6b27fd8db 100644
--- a/gdb/testsuite/gdb.cp/gdb2384.exp
+++ b/gdb/testsuite/gdb.cp/gdb2384.exp
@@ -25,6 +25,8 @@ require allow_cplus_tests allow_shlib_tests
standard_testfile .cc gdb2384-base.cc
+include_file gdb2384-base.h
+
set sofile [standard_output_file gdb2384-base.so]
if { [gdb_compile_shlib $srcdir/$subdir/$srcfile2 $sofile {debug c++}] != ""
diff --git a/gdb/testsuite/gdb.cp/hang.exp b/gdb/testsuite/gdb.cp/hang.exp
index 7d8588520ec..71ab5c7a63b 100644
--- a/gdb/testsuite/gdb.cp/hang.exp
+++ b/gdb/testsuite/gdb.cp/hang.exp
@@ -18,6 +18,8 @@ require allow_cplus_tests
standard_testfile hang1.cc hang2.cc hang3.cc
+include_file hang.H
+
if {[prepare_for_testing "failed to prepare" $testfile \
[list $srcfile $srcfile2 $srcfile3] {debug c++}]} {
return -1
diff --git a/gdb/testsuite/gdb.cp/m-static.exp b/gdb/testsuite/gdb.cp/m-static.exp
index 56c4cdad95e..419d772e28e 100644
--- a/gdb/testsuite/gdb.cp/m-static.exp
+++ b/gdb/testsuite/gdb.cp/m-static.exp
@@ -27,6 +27,8 @@ require allow_cplus_tests
standard_testfile .cc m-static1.cc
+include_file m-static.h
+
set flags [list debug c++]
if { [test_compiler_info gcc-*] && [gcc_major_version] >= 10 } {
# Work around PR gcc/101452.
@@ -181,7 +183,11 @@ gdb_test "print test4.somewhere" "\\$\[0-9\].* = 3.14\[0-9\]*" "static const flo
# Also make sure static const members can be found via "info var".
if { $non_dwarf } { setup_xfail *-*-* }
-gdb_test "info variable everywhere" "File .*/m-static\[.\]h.*const int gnu_obj_4::everywhere;"
+gdb_test "info variable everywhere" \
+ [multi_line \
+ "File (.*/)?m-static\[.\]h:" \
+ "$decimal:\tconst int gnu_obj_4::everywhere;" \
+ ".*"]
# Perhaps at some point test4 should also include a test for a static
# const int that was initialized in the header file. But I'm not sure
diff --git a/gdb/testsuite/gdb.cp/mb-inline.exp b/gdb/testsuite/gdb.cp/mb-inline.exp
index 555827806f7..9793b14c073 100644
--- a/gdb/testsuite/gdb.cp/mb-inline.exp
+++ b/gdb/testsuite/gdb.cp/mb-inline.exp
@@ -22,6 +22,7 @@ require allow_cplus_tests
standard_testfile mb-inline1.cc mb-inline2.cc
set hdrfile "${testfile}.h"
+include_file $hdrfile
if {[prepare_for_testing "failed to prepare" $testfile \
[list $srcfile $srcfile2] {debug c++}]} {
diff --git a/gdb/testsuite/gdb.cp/minsym-fallback.exp b/gdb/testsuite/gdb.cp/minsym-fallback.exp
index 56a56a3da6b..a3bfe56bec6 100644
--- a/gdb/testsuite/gdb.cp/minsym-fallback.exp
+++ b/gdb/testsuite/gdb.cp/minsym-fallback.exp
@@ -18,6 +18,9 @@
# a different file providing only ELF symbols.
standard_testfile .cc minsym-fallback-main.cc
+
+include_file minsym-fallback.h
+
set executable $testfile
set objfile [standard_output_file ${testfile}.o]
set objmainfile [standard_output_file ${testfile}-main.o]
diff --git a/gdb/testsuite/gdb.cp/ovsrch.exp b/gdb/testsuite/gdb.cp/ovsrch.exp
index 31a8abf7793..39e022c6676 100644
--- a/gdb/testsuite/gdb.cp/ovsrch.exp
+++ b/gdb/testsuite/gdb.cp/ovsrch.exp
@@ -57,6 +57,8 @@ require allow_cplus_tests
# Test for c++/11734
standard_testfile ovsrch1.cc ovsrch2.cc ovsrch3.cc ovsrch4.cc
+include_file ovsrch.h
+
if {[prepare_for_testing "failed to prepare" $testfile \
[list $srcfile $srcfile2 $srcfile3 $srcfile4] {c++ debug}]} {
return -1
diff --git a/gdb/testsuite/gdb.cp/pr10728.exp b/gdb/testsuite/gdb.cp/pr10728.exp
index b860fb7c3ae..ea9c1242725 100644
--- a/gdb/testsuite/gdb.cp/pr10728.exp
+++ b/gdb/testsuite/gdb.cp/pr10728.exp
@@ -22,6 +22,8 @@ require allow_cplus_tests
load_lib "cp-support.exp"
standard_testfile pr10728-x.cc pr10728-y.cc
+include_file pr10728-x.h
+
set tfx [standard_output_file pr10728-x.o]
set tfy [standard_output_file pr10728-y.o]
diff --git a/gdb/testsuite/gdb.cp/rtti.exp b/gdb/testsuite/gdb.cp/rtti.exp
index 8ad7683ee40..18dfaf0c2bd 100644
--- a/gdb/testsuite/gdb.cp/rtti.exp
+++ b/gdb/testsuite/gdb.cp/rtti.exp
@@ -34,6 +34,8 @@ require allow_cplus_tests
standard_testfile rtti1.cc rtti2.cc
+include_file rtti.h
+
if {[prepare_for_testing "failed to prepare" $testfile \
[list $srcfile $srcfile2] {debug c++}]} {
return -1
diff --git a/gdb/testsuite/gdb.cp/stub-array-size.exp b/gdb/testsuite/gdb.cp/stub-array-size.exp
index 0afe232d583..7757db20af9 100644
--- a/gdb/testsuite/gdb.cp/stub-array-size.exp
+++ b/gdb/testsuite/gdb.cp/stub-array-size.exp
@@ -22,6 +22,8 @@ require allow_cplus_tests
standard_testfile .cc stub-array-size2.cc
+include_file stub-array-size.h
+
if {[prepare_for_testing "failed to prepare" $testfile "$srcfile $srcfile2" \
{c++ debug}]} {
return -1
diff --git a/gdb/testsuite/gdb.cp/subtypes.exp b/gdb/testsuite/gdb.cp/subtypes.exp
index f5cf349cc26..afad478cba3 100644
--- a/gdb/testsuite/gdb.cp/subtypes.exp
+++ b/gdb/testsuite/gdb.cp/subtypes.exp
@@ -22,6 +22,8 @@ load_lib "cp-support.exp"
standard_testfile .cc subtypes-2.cc
+include_file subtypes.h
+
if {[prepare_for_testing "failed to prepare" $testfile \
[list $srcfile $srcfile2] \
{debug c++ additional_flags=-std=c++11}]} {
diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp
index 0a0ae697d1e..24fcb8f1746 100644
--- a/gdb/testsuite/lib/gdb.exp
+++ b/gdb/testsuite/lib/gdb.exp
@@ -9554,6 +9554,18 @@ gdb_caching_proc have_compile_flag { flag } {
additional_flags=$flag]
}
+# Handle include file $srcdir/$subdir/FILE.
+
+proc include_file { file } {
+ set file [file join $::srcdir $::subdir $file]
+ if { [is_remote host] } {
+ set res [remote_download host $file]
+ } else {
+ set res $file
+ }
+
+ return $res
+}
# Handle include file FILE, and if necessary update compiler flags variable
# FLAGS.
--
2.35.3
^ permalink raw reply [flat|nested] 7+ messages in thread
* [pushed 2/3] [gdb/testsuite] Fix gdb.cp/annota{2,3}.exp for native-extended-gdbserver
2023-03-22 8:38 [pushed 0/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host/target Tom de Vries
2023-03-22 8:38 ` [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host Tom de Vries
@ 2023-03-22 8:38 ` Tom de Vries
2023-03-22 8:38 ` [pushed 3/3] [gdb/testsuite] Fix gdb.cp/cp-relocate.exp for remote host Tom de Vries
2 siblings, 0 replies; 7+ messages in thread
From: Tom de Vries @ 2023-03-22 8:38 UTC (permalink / raw)
To: gdb-patches
When running test-cases gdb.cp/annota{2,3}.exp with target board
native-extended-gdbserver, we run into a few FAILs, due to the test-cases
trying to match inferior output together with gdb output.
Fix this by ignoring the inferior output in this case.
Tested on x86_64-linux.
---
gdb/testsuite/gdb.cp/annota2.exp | 15 +++++++++++++--
gdb/testsuite/gdb.cp/annota3.exp | 25 +++++++++++++++++--------
2 files changed, 30 insertions(+), 10 deletions(-)
diff --git a/gdb/testsuite/gdb.cp/annota2.exp b/gdb/testsuite/gdb.cp/annota2.exp
index 64adbbdb220..506817bd498 100644
--- a/gdb/testsuite/gdb.cp/annota2.exp
+++ b/gdb/testsuite/gdb.cp/annota2.exp
@@ -126,13 +126,24 @@ gdb_test_multiple "print a" "print class" {
# `a.x is 1' is asynchronous regarding to `frames-invalid'.
#
+if { $gdb_spawn_id == $inferior_spawn_id } {
+ set match_output \
+ [multi_line \
+ "\($frames_invalid\)*a.x is 1" \
+ "\($frames_invalid\)*"]
+} else {
+ # Usually we'd handle this using gdb_test_stdio, but that looks too
+ # complicated in combination with annotations, so just ignore the inferior
+ # output.
+ set match_output "\($frames_invalid\)*"
+}
+
set pat [multi_line "" \
"\032\032post-prompt" \
"Continuing." \
"" \
"\032\032starting" \
- "\(${frames_invalid}\)*a.x is 1" \
- "\(${frames_invalid}\)*"\
+ $match_output \
"\032\032exited 0" \
"$inferior_exited_re normally." \
"" \
diff --git a/gdb/testsuite/gdb.cp/annota3.exp b/gdb/testsuite/gdb.cp/annota3.exp
index 788bda77f3d..7b85c84cd97 100644
--- a/gdb/testsuite/gdb.cp/annota3.exp
+++ b/gdb/testsuite/gdb.cp/annota3.exp
@@ -95,14 +95,23 @@ gdb_expect_list "print class" "$gdb_prompt$" {
# annotate-exited
#
send_gdb "continue\n"
-gdb_expect_list "continue to exit" "$gdb_prompt$" [concat {
- "\r\n\032\032post-prompt\r\n"
- "Continuing.\r\n"
- "\r\n\032\032starting\r\n"
- "a.x is 1\r\n"
- "\r\n\032\032exited 0\r\n"} [list "$inferior_exited_re normally.\r\n"] {
- "\r\n\032\032stopped\r\n"
-}]
+
+set el {}
+lappend el "\r\n\032\032post-prompt\r\n"
+lappend el "Continuing.\r\n"
+lappend el "\r\n\032\032starting\r\n"
+if { $gdb_spawn_id == $inferior_spawn_id } {
+ lappend el "a.x is 1\r\n"
+} else {
+ # Usually we'd handle this using gdb_test_stdio, but that looks too
+ # complicated in combination with annotations, so just ignore the inferior
+ # output.
+}
+lappend el "\r\n\032\032exited 0\r\n"
+lappend el "$inferior_exited_re normally.\r\n"
+lappend el "\r\n\032\032stopped\r\n"
+
+gdb_expect_list "continue to exit" "$gdb_prompt$" $el
#
# delete all breakpoints
--
2.35.3
^ permalink raw reply [flat|nested] 7+ messages in thread
* [pushed 3/3] [gdb/testsuite] Fix gdb.cp/cp-relocate.exp for remote host
2023-03-22 8:38 [pushed 0/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host/target Tom de Vries
2023-03-22 8:38 ` [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host Tom de Vries
2023-03-22 8:38 ` [pushed 2/3] [gdb/testsuite] Fix gdb.cp/annota{2,3}.exp for native-extended-gdbserver Tom de Vries
@ 2023-03-22 8:38 ` Tom de Vries
2 siblings, 0 replies; 7+ messages in thread
From: Tom de Vries @ 2023-03-22 8:38 UTC (permalink / raw)
To: gdb-patches
Fix test-case gdb.cp/cp-relocate.exp for remote host using
gdb_remote_download.
Tested on x86_64-linux.
---
gdb/testsuite/gdb.cp/cp-relocate.exp | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/gdb/testsuite/gdb.cp/cp-relocate.exp b/gdb/testsuite/gdb.cp/cp-relocate.exp
index d400bb14e46..25ed15a9aa1 100644
--- a/gdb/testsuite/gdb.cp/cp-relocate.exp
+++ b/gdb/testsuite/gdb.cp/cp-relocate.exp
@@ -118,7 +118,9 @@ if { $func1_sec == $func2_sec } {
# Now start a clean GDB, for add-symbol-file tests.
clean_restart
-gdb_test "add-symbol-file ${binfile} 0 -s ${func1_sec} 0x10000 -s ${func2_sec} 0x20000" \
+set host_binfile [gdb_remote_download host $binfile]
+
+gdb_test "add-symbol-file $host_binfile 0 -s ${func1_sec} 0x10000 -s ${func2_sec} 0x20000" \
"Reading symbols from .*${testfile}\\.o\\.\\.\\.(|\r\nUsing host libthread_db library .*libthread_db.so.*\\.)" \
"add-symbol-file ${testfile}.o" \
"add symbol table from file \".*${testfile}\\.o\" at.*\\(y or n\\) " \
--
2.35.3
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host
2023-03-22 8:38 ` [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host Tom de Vries
@ 2023-03-22 15:05 ` Simon Marchi
2023-03-23 0:18 ` Tom de Vries
0 siblings, 1 reply; 7+ messages in thread
From: Simon Marchi @ 2023-03-22 15:05 UTC (permalink / raw)
To: Tom de Vries, gdb-patches
On 3/22/23 04:38, Tom de Vries via Gdb-patches wrote:
> Fix a few test-cases in gdb.cp/*.exp for remote host using new proc
> include_file.
Starting with this patch, I see:
info variable everywhere^M
All variables matching regular expression "everywhere":^M
^M
File /home/simark/src/binutils-gdb/gdb/testsuite/gdb.cp/m-static.h:^M
8: const int gnu_obj_4::everywhere;^M
(gdb) FAIL: gdb.cp/m-static.exp: info variable everywhere
Simon
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host
2023-03-22 15:05 ` Simon Marchi
@ 2023-03-23 0:18 ` Tom de Vries
2023-03-23 13:49 ` Tom de Vries
0 siblings, 1 reply; 7+ messages in thread
From: Tom de Vries @ 2023-03-23 0:18 UTC (permalink / raw)
To: Simon Marchi, gdb-patches
On 3/22/23 16:05, Simon Marchi wrote:
>
>
> On 3/22/23 04:38, Tom de Vries via Gdb-patches wrote:
>> Fix a few test-cases in gdb.cp/*.exp for remote host using new proc
>> include_file.
>
> Starting with this patch, I see:
>
> info variable everywhere^M
> All variables matching regular expression "everywhere":^M
> ^M
> File /home/simark/src/binutils-gdb/gdb/testsuite/gdb.cp/m-static.h:^M
> 8: const int gnu_obj_4::everywhere;^M
> (gdb) FAIL: gdb.cp/m-static.exp: info variable everywhere
>
Hi Simon,
thanks for reporting this.
I don't see this on openSUSE Leap 15.4 and Tumbleweed, but I managed to
reproduce on Ubuntu 20.04 (though with the readnow board, it disappears
again).
Anyway, the problem is that I changed the matching to be more strict,
and that's what we're seeing the effect of. I'll propose a patch that
reverts things to the way things were before, with less strict matching.
As for what I'm seeing:
...
(gdb) info variable everywhere^M
All variables matching regular expression "everywhere":^M
^M
File /data/vries/gdb/src/gdb/testsuite/gdb.cp/m-static.h:^M
8: const int gnu_obj_4::everywhere;^M
^M
File /data/vries/gdb/src/gdb/testsuite/gdb.cp/m-static1.cc:^M
8: const int gnu_obj_4::everywhere;^M
(gdb) PASS: gdb.cp/m-static.exp: info variable everywhere
...
the combination of line 8 with m-static1.cc doesn't seem to make sense.
I don't know this as a known issue, so I'll file a PR about it.
Thanks,
- Tom
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host
2023-03-23 0:18 ` Tom de Vries
@ 2023-03-23 13:49 ` Tom de Vries
0 siblings, 0 replies; 7+ messages in thread
From: Tom de Vries @ 2023-03-23 13:49 UTC (permalink / raw)
To: Simon Marchi, gdb-patches
On 3/23/23 01:18, Tom de Vries wrote:
> On 3/22/23 16:05, Simon Marchi wrote:
>>
>>
>> On 3/22/23 04:38, Tom de Vries via Gdb-patches wrote:
>>> Fix a few test-cases in gdb.cp/*.exp for remote host using new proc
>>> include_file.
>>
>> Starting with this patch, I see:
>>
>> info variable everywhere^M
>> All variables matching regular expression "everywhere":^M
>> ^M
>> File
>> /home/simark/src/binutils-gdb/gdb/testsuite/gdb.cp/m-static.h:^M
>> 8: const int gnu_obj_4::everywhere;^M
>> (gdb) FAIL: gdb.cp/m-static.exp: info variable everywhere
>>
>
> Hi Simon,
>
> thanks for reporting this.
>
> I don't see this on openSUSE Leap 15.4 and Tumbleweed, but I managed to
> reproduce on Ubuntu 20.04 (though with the readnow board, it disappears
> again).
>
> Anyway, the problem is that I changed the matching to be more strict,
> and that's what we're seeing the effect of. I'll propose a patch that
> reverts things to the way things were before, with less strict matching.
>
Proposed patch is here (
https://sourceware.org/pipermail/gdb-patches/2023-March/198240.html ).
> As for what I'm seeing:
> ...
> (gdb) info variable everywhere^M
> All variables matching regular expression "everywhere":^M
> ^M
> File /data/vries/gdb/src/gdb/testsuite/gdb.cp/m-static.h:^M
> 8: const int gnu_obj_4::everywhere;^M
> ^M
> File /data/vries/gdb/src/gdb/testsuite/gdb.cp/m-static1.cc:^M
> 8: const int gnu_obj_4::everywhere;^M
> (gdb) PASS: gdb.cp/m-static.exp: info variable everywhere
> ...
> the combination of line 8 with m-static1.cc doesn't seem to make sense.
>
> I don't know this as a known issue, so I'll file a PR about it.
>
Filed this PR ( https://sourceware.org/bugzilla/show_bug.cgi?id=30265 ).
Thanks,
- Tom
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2023-03-23 13:49 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-22 8:38 [pushed 0/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host/target Tom de Vries
2023-03-22 8:38 ` [pushed 1/3] [gdb/testsuite] Fix gdb.cp/*.exp for remote host Tom de Vries
2023-03-22 15:05 ` Simon Marchi
2023-03-23 0:18 ` Tom de Vries
2023-03-23 13:49 ` Tom de Vries
2023-03-22 8:38 ` [pushed 2/3] [gdb/testsuite] Fix gdb.cp/annota{2,3}.exp for native-extended-gdbserver Tom de Vries
2023-03-22 8:38 ` [pushed 3/3] [gdb/testsuite] Fix gdb.cp/cp-relocate.exp for remote host Tom de Vries
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).