From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 7876) id 1CF0C395A027; Tue, 31 May 2022 14:46:04 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 1CF0C395A027 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Nils-Christian Kempke To: gdb-cvs@sourceware.org Subject: [binutils-gdb] testsuite/lib: add check_optional_entry for GDBInfoSymbols X-Act-Checkin: binutils-gdb X-Git-Author: Nils-Christian Kempke X-Git-Refname: refs/heads/master X-Git-Oldrev: 4212a8c90777ba7c5fdc09d4022a1fb353851b4f X-Git-Newrev: 5c8ff7f6d9240026ef7d87e3efae5b127fdba365 Message-Id: <20220531144604.1CF0C395A027@sourceware.org> Date: Tue, 31 May 2022 14:46:04 +0000 (GMT) X-BeenThere: gdb-cvs@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-cvs mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 31 May 2022 14:46:04 -0000 https://sourceware.org/git/gitweb.cgi?p=3Dbinutils-gdb.git;h=3D5c8ff7f6d924= 0026ef7d87e3efae5b127fdba365 commit 5c8ff7f6d9240026ef7d87e3efae5b127fdba365 Author: Nils-Christian Kempke Date: Tue May 31 16:43:44 2022 +0200 testsuite/lib: add check_optional_entry for GDBInfoSymbols =20 There was already a similar functionality for the GDBInfoModuleSymbols. This just extends the GDBInfoSymbols. We will use this feature in a later commit to make a testcase less GNU specific and more flexible for other compilers. =20 Namely, in gdb.fortran/info-types.exp currenlty GDBInfoSymbols::check_entry is used to verify and test the output of the info symbols command. The test, however was written with gfortran as a basis and some of the tests are not fair with e.g. ifx and ifort as they test for symbols that are not actually required to be emitted. The lines GDBInfoSymbols::check_entry "${srcfile}" "" "${character1}" and GDBInfoSymbols::check_entry "${srcfile}" "37" "Type s1;" =20 check for types that are either not used in the source file (character1) or should not be emitted by the compiler at global scope (s1) thus no appearing in the info symbols command. In order to fix this we will later use the newly introduced check_optional_entry over check_entry. Diff: --- gdb/testsuite/lib/sym-info-cmds.exp | 18 ++++++++++++++++-- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/gdb/testsuite/lib/sym-info-cmds.exp b/gdb/testsuite/lib/sym-in= fo-cmds.exp index f0272a03349..ce878f0d502 100644 --- a/gdb/testsuite/lib/sym-info-cmds.exp +++ b/gdb/testsuite/lib/sym-info-cmds.exp @@ -163,6 +163,16 @@ namespace eval GDBInfoSymbols { gdb_assert {[regexp -- $pattern $_header]} $testname } =20 + # Call check_entry_1 with OPTIONAL =3D=3D 0. + proc check_entry { filename lineno text { testname "" } } { + check_entry_1 $filename $lineno $text 0 $testname + } + + # Call check_entry_1 with OPTIONAL =3D=3D 1. + proc check_optional_entry { filename lineno text { testname "" } } { + check_entry_1 $filename $lineno $text 1 $testname + } + # Check that we have an entry in _entries matching FILENAME, # LINENO, and TEXT. If LINENO is the empty string it is replaced # with the string NONE in order to match a similarly missing line @@ -174,7 +184,7 @@ namespace eval GDBInfoSymbols { # If a matching entry is found then it is removed from the # _entries list, this allows us to check for duplicates using the # check_no_entry call. - proc check_entry { filename lineno text { testname "" } } { + proc check_entry_1 { filename lineno text optional testname } { variable _entries variable _last_command =20 @@ -209,7 +219,11 @@ namespace eval GDBInfoSymbols { } =20 set _entries $new_entries - gdb_assert { $found_match } $testname + if { $optional && ! $found_match } { + unsupported $testname + } else { + gdb_assert { $found_match } $testname + } } =20 # Check that there is no entry in the _entries list matching