From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx2.suse.de (mx2.suse.de [195.135.220.15]) by sourceware.org (Postfix) with ESMTPS id 797603857C6E for ; Fri, 17 Jul 2020 11:07:10 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 797603857C6E Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=suse.de Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=tdevries@suse.de X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.221.27]) by mx2.suse.de (Postfix) with ESMTP id D3947AE3C for ; Fri, 17 Jul 2020 11:07:13 +0000 (UTC) Date: Fri, 17 Jul 2020 13:07:07 +0200 From: Tom de Vries To: gdb-patches@sourceware.org Subject: [committed][gdb/testsuite] Remove Dwarf::extern Message-ID: <20200717110706.GA5585@delia> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.10.1 (2018-07-13) X-Spam-Status: No, score=-11.1 required=5.0 tests=BAYES_00, GIT_PATCH_0, KAM_DMARC_STATUS, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 17 Jul 2020 11:07:12 -0000 Hi, The file lib/dwarf.exp contains: ... # Declare a global label. This is typically used to refer to # labels defined in other files, for example a function defined in # a .c file. proc extern {args} { foreach name $args { _op .global $name } } ... The assembler directive to refer to labels defined in other files is not .global, but .extern, and that one is ignored by gas. Since we require gas for all dwarf assembly test-cases, remove the proc and all it's uses. Tested on x86_64-linux. Committed to trunk. Thanks, - Tom [gdb/testsuite] Remove Dwarf::extern gdb/testsuite/ChangeLog: 2020-07-17 Tom de Vries * lib/dwarf.exp (Dwarf::extern): Remove. * gdb.compile/compile-ops.exp: Remove use of Dwarf::extern. * gdb.dlang/circular.exp: Same. * gdb.dwarf2/comp-unit-lang.exp: Same. * gdb.dwarf2/dw2-ifort-parameter.exp: Same. * gdb.dwarf2/dw2-symtab-includes.exp: Same. * gdb.dwarf2/dwz.exp: Same. * gdb.dwarf2/imported-unit-abstract-const-value.exp: Same. * gdb.dwarf2/imported-unit-runto-main.exp: Same. * gdb.dwarf2/imported-unit.exp: Same. * gdb.dwarf2/opaque-type-lookup.exp: Same. --- gdb/testsuite/gdb.compile/compile-ops.exp | 1 - gdb/testsuite/gdb.dlang/circular.exp | 2 -- gdb/testsuite/gdb.dwarf2/comp-unit-lang.exp | 2 -- gdb/testsuite/gdb.dwarf2/dw2-ifort-parameter.exp | 1 - gdb/testsuite/gdb.dwarf2/dw2-symtab-includes.exp | 2 -- gdb/testsuite/gdb.dwarf2/dwz.exp | 2 -- gdb/testsuite/gdb.dwarf2/imported-unit-abstract-const-value.exp | 2 -- gdb/testsuite/gdb.dwarf2/imported-unit-runto-main.exp | 2 -- gdb/testsuite/gdb.dwarf2/imported-unit.exp | 3 --- gdb/testsuite/gdb.dwarf2/opaque-type-lookup.exp | 2 -- gdb/testsuite/lib/dwarf.exp | 9 --------- 11 files changed, 28 deletions(-) diff --git a/gdb/testsuite/gdb.compile/compile-ops.exp b/gdb/testsuite/gdb.compile/compile-ops.exp index d9aee413a6..7c801d932c 100644 --- a/gdb/testsuite/gdb.compile/compile-ops.exp +++ b/gdb/testsuite/gdb.compile/compile-ops.exp @@ -362,7 +362,6 @@ Dwarf::assemble $asm_file { cu { addr_size 4 } { declare_labels int_label - extern func_start func_end ptr compile_unit { {name file1.txt} diff --git a/gdb/testsuite/gdb.dlang/circular.exp b/gdb/testsuite/gdb.dlang/circular.exp index f3d8f60469..1e04370478 100644 --- a/gdb/testsuite/gdb.dlang/circular.exp +++ b/gdb/testsuite/gdb.dlang/circular.exp @@ -37,8 +37,6 @@ Dwarf::assemble $asm_file { declare_labels circular1_label circular2_label circular3_label declare_labels circular4_label circular5_label - extern circular1_found_start circular1_found_end - circular1_label: module { {name circular1} } { diff --git a/gdb/testsuite/gdb.dwarf2/comp-unit-lang.exp b/gdb/testsuite/gdb.dwarf2/comp-unit-lang.exp index 8ff6b733cc..2ebe6caedc 100644 --- a/gdb/testsuite/gdb.dwarf2/comp-unit-lang.exp +++ b/gdb/testsuite/gdb.dwarf2/comp-unit-lang.exp @@ -42,8 +42,6 @@ proc do_test {cu_lang gdb_lang} { # Creating a CU with 4-byte addresses lets this test link on # both 32- and 64-bit machines. cu { addr_size 4 } { - extern func_start func_end - compile_unit { {name file1.txt} {language @$cu_lang} diff --git a/gdb/testsuite/gdb.dwarf2/dw2-ifort-parameter.exp b/gdb/testsuite/gdb.dwarf2/dw2-ifort-parameter.exp index 75f17c5192..2388ce53f6 100644 --- a/gdb/testsuite/gdb.dwarf2/dw2-ifort-parameter.exp +++ b/gdb/testsuite/gdb.dwarf2/dw2-ifort-parameter.exp @@ -29,7 +29,6 @@ set asm_file [standard_output_file $srcfile2] Dwarf::assemble $asm_file { declare_labels int_label - extern func_start func_end ptr global srcdir subdir srcfile cu {} { diff --git a/gdb/testsuite/gdb.dwarf2/dw2-symtab-includes.exp b/gdb/testsuite/gdb.dwarf2/dw2-symtab-includes.exp index 1eaaf4af4f..e10e413035 100644 --- a/gdb/testsuite/gdb.dwarf2/dw2-symtab-includes.exp +++ b/gdb/testsuite/gdb.dwarf2/dw2-symtab-includes.exp @@ -31,8 +31,6 @@ Dwarf::assemble $asm_file { declare_labels partial_label lines_label global srcdir subdir srcfile - extern main - cu {} { partial_label: partial_unit {} { } diff --git a/gdb/testsuite/gdb.dwarf2/dwz.exp b/gdb/testsuite/gdb.dwarf2/dwz.exp index b9987ed0d1..9f5eda5e38 100644 --- a/gdb/testsuite/gdb.dwarf2/dwz.exp +++ b/gdb/testsuite/gdb.dwarf2/dwz.exp @@ -28,8 +28,6 @@ Dwarf::assemble $asm_file { declare_labels partial_label int_label int_label2 global srcdir subdir srcfile - extern main - cu {} { partial_label: partial_unit {} { subprogram { diff --git a/gdb/testsuite/gdb.dwarf2/imported-unit-abstract-const-value.exp b/gdb/testsuite/gdb.dwarf2/imported-unit-abstract-const-value.exp index f93dc12388..13a7e97cc5 100644 --- a/gdb/testsuite/gdb.dwarf2/imported-unit-abstract-const-value.exp +++ b/gdb/testsuite/gdb.dwarf2/imported-unit-abstract-const-value.exp @@ -42,8 +42,6 @@ Dwarf::assemble $asm_file { global srcdir subdir srcfile - extern main - set main_range [function_range main ${srcdir}/${subdir}/${srcfile}] set main_start [lindex $main_range 0] set main_length [lindex $main_range 1] diff --git a/gdb/testsuite/gdb.dwarf2/imported-unit-runto-main.exp b/gdb/testsuite/gdb.dwarf2/imported-unit-runto-main.exp index 2794684053..8a05ee8527 100644 --- a/gdb/testsuite/gdb.dwarf2/imported-unit-runto-main.exp +++ b/gdb/testsuite/gdb.dwarf2/imported-unit-runto-main.exp @@ -39,8 +39,6 @@ Dwarf::assemble $asm_file { global srcdir subdir srcfile - extern main - set main_range [function_range main ${srcdir}/${subdir}/${srcfile}] set main_start [lindex $main_range 0] set main_length [lindex $main_range 1] diff --git a/gdb/testsuite/gdb.dwarf2/imported-unit.exp b/gdb/testsuite/gdb.dwarf2/imported-unit.exp index 32a9abf620..dba87ffcb2 100644 --- a/gdb/testsuite/gdb.dwarf2/imported-unit.exp +++ b/gdb/testsuite/gdb.dwarf2/imported-unit.exp @@ -56,9 +56,6 @@ Dwarf::assemble $asm_file { global srcdir subdir srcfile - extern main - extern doit - set main_range [function_range main ${srcdir}/${subdir}/${srcfile}] set main_start [lindex $main_range 0] set main_length [lindex $main_range 1] diff --git a/gdb/testsuite/gdb.dwarf2/opaque-type-lookup.exp b/gdb/testsuite/gdb.dwarf2/opaque-type-lookup.exp index 60d67b1bda..19316ef7eb 100644 --- a/gdb/testsuite/gdb.dwarf2/opaque-type-lookup.exp +++ b/gdb/testsuite/gdb.dwarf2/opaque-type-lookup.exp @@ -34,8 +34,6 @@ Dwarf::assemble $asm_file { declare_labels char_type1_label char_type2_label global srcdir subdir srcfile - extern main - # The partial units are laid out so we're not dependent on the order that # they appear in compunit_symtab.includes. We need the one with the # opaque definition to appear first to gdb, so we put it in the middle. diff --git a/gdb/testsuite/lib/dwarf.exp b/gdb/testsuite/lib/dwarf.exp index 21dc0f62d4..f745b2ae31 100644 --- a/gdb/testsuite/lib/dwarf.exp +++ b/gdb/testsuite/lib/dwarf.exp @@ -822,15 +822,6 @@ namespace eval Dwarf { _emit "${name}:" } - # Declare a global label. This is typically used to refer to - # labels defined in other files, for example a function defined in - # a .c file. - proc extern {args} { - foreach name $args { - _op .global $name - } - } - # A higher-level interface to label handling. # # ARGS is a list of label descriptors. Each one is either a