From patchwork Sat Oct 24 15:28:08 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "H.J. Lu" X-Patchwork-Id: 1387063 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=gcc.gnu.org (client-ip=2620:52:3:1:0:246e:9693:128c; helo=sourceware.org; envelope-from=gcc-patches-bounces@gcc.gnu.org; receiver=) Authentication-Results: ozlabs.org; dmarc=pass (p=none dis=none) header.from=gcc.gnu.org Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=gcc.gnu.org header.i=@gcc.gnu.org header.a=rsa-sha256 header.s=default header.b=LAgbEpJJ; dkim-atps=neutral Received: from sourceware.org (server2.sourceware.org [IPv6:2620:52:3:1:0:246e:9693:128c]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 4CJQ465Vmnz9sSG for ; Sun, 25 Oct 2020 02:28:52 +1100 (AEDT) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 28E8B385483D; Sat, 24 Oct 2020 15:28:50 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 28E8B385483D DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1603553330; bh=/IPLHTOOamRyoO2GTPqeprWis9DQXGI6HBpz18AYaWA=; h=References:In-Reply-To:Date:Subject:To:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc: From; b=LAgbEpJJYWmFZjBI955nlv0sSjQC223FHJ6uVpv78uim7LNzGlovlkz3y+9BA4rLx pVVo8AHGXXrgHlk30iTbBrW77oWJ+MOtwFU4lL7zqVytyA/rOgU9ldWMc7dugEKysa 6kGiCUOOK8xdbPWzSQoay25+w8NRFI622uOr7hEw= X-Original-To: gcc-patches@gcc.gnu.org Delivered-To: gcc-patches@gcc.gnu.org Received: from mail-oi1-x22a.google.com (mail-oi1-x22a.google.com [IPv6:2607:f8b0:4864:20::22a]) by sourceware.org (Postfix) with ESMTPS id 396383854837 for ; Sat, 24 Oct 2020 15:28:45 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 396383854837 Received: by mail-oi1-x22a.google.com with SMTP id x203so414607oia.10 for ; Sat, 24 Oct 2020 08:28:45 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=/IPLHTOOamRyoO2GTPqeprWis9DQXGI6HBpz18AYaWA=; b=ZwwTpvnqcZPZQWi9tzr++soN4NTkfESvpop5mGd7i7fU4gmecXYhmNb8lvdJfpCKAY sfMmSVng++hJS5rp2+z4k60Rd58aa2rNdldh0qf0rcdCbJWoDB6qDZGvEUPhpuIH4emH sV1gZGCLAbekd50qHEIEgwmtBmZDxeRUWKS0PiVT3tWTEalWP8be+3hvdB7l8IkYwfoM ZFVJo5ZgujZemNVw17b0Z7/1s3tZnOqwqW7VHtYv9MRnQoPq8iNGIDB2MLgI6QKsvZTk UB1/3Jd4qDIWW4B0N+9tGdV66ht8MFs1NWrpYi+w2JPk8+NC5EifR2vvixVNimSq9RNX tySQ== X-Gm-Message-State: AOAM532HnHxsHv1kILdvikfbcB4YnJ61pAKULvhhNDRbXxWmyasTAHJ8 fTZqXwoitani3jp2f9X9O+FZdsYUqkcfxHrGPoI= X-Google-Smtp-Source: ABdhPJz2Cv0CPOZ75/10KFqKcwLv1Lf8BHZPWIRYG9YRLGBfzHVSxSqG4ZCOk0klS05FORnOMaDDVuj45wXSabcSYco= X-Received: by 2002:aca:5605:: with SMTP id k5mr5172254oib.25.1603553324625; Sat, 24 Oct 2020 08:28:44 -0700 (PDT) MIME-Version: 1.0 References: <20201016175834.2289120-1-hjl.tools@gmail.com> <20201016181703.GH2176@tucnak> In-Reply-To: Date: Sat, 24 Oct 2020 08:28:08 -0700 Message-ID: Subject: V2 [PATCH] Update check for working assembler --gdwarf-4 option To: Jakub Jelinek X-Spam-Status: No, score=-3038.3 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, FREEMAIL_FROM, GIT_PATCH_0, RCVD_IN_DNSWL_NONE, 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: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-Patchwork-Original-From: "H.J. Lu via Gcc-patches" From: "H.J. Lu" Reply-To: "H.J. Lu" Cc: GCC Patches Errors-To: gcc-patches-bounces@gcc.gnu.org Sender: "Gcc-patches" On Sat, Oct 17, 2020 at 6:23 AM H.J. Lu wrote: > > On Fri, Oct 16, 2020 at 11:29 AM H.J. Lu wrote: > > > > On Fri, Oct 16, 2020 at 11:17 AM Jakub Jelinek wrote: > > > > > > On Fri, Oct 16, 2020 at 10:58:34AM -0700, H.J. Lu wrote: > > > > Don't set HAVE_AS_GDWARF_5_DEBUG_FLAG nor HAVE_AS_WORKING_DWARF_4_FLAG > > > > if there is an extra assembly input file in debug info generated by > > > > --gdwarf-5/--gdwarf-4: > > > > > > > > https://sourceware.org/bugzilla/show_bug.cgi?id=25878 > > > > https://sourceware.org/bugzilla/show_bug.cgi?id=26740 > > > > > > > > Also replace success with dwarf4_success in the 32-bit --gdwarf-4 check. > > > > > > > > OK for master? > > > > > > > > PR bootstrap/97451 > > > > * configure.ac (HAVE_AS_GDWARF_5_DEBUG_FLAG): Don't define if > > > > there is an extra assembly input file in debug info. > > > > (HAVE_AS_WORKING_DWARF_4_FLAG): Likewise. Replace success with > > > > dwarf4_success in the 32-bit --gdwarf-4 check. > > > > * configure: Regenerated. > > > > > > The HAVE_AS_GDWARF_5_DEBUG_FLAG macro should be solely about whether > > > -gdwarf-5 can be passed to as, nothing else. That is because it is not only > > > used to decide if we can pass -gdwarf-5 to as for assembly of e.g. *.c > > > compilation, but also if we can pass it to as when gcc driver is invoked on > > > *.s and *.S. And in that case the problems with -gdwarf-N having unwanted > > > effects on assembly with compiler generated .debug_info etc. sections. > > > don't really matter. > > > > > > The HAVE_AS_WORKING_DWARF_4_FLAG perhaps could be renamed to > > > ...DWARF_N_FLAG, it is meant to check for whether we can safely pass the > > > option also for *.c etc. compilation when we emit debug info sections by the > > > compiler. > > > > > > For that, the question is, are all the issues already fixed on the binutils > > > trunk? As in, do I get identical object files e.g. for -gdwarf-2 > > > > Not yet. This patch: > > > > https://sourceware.org/pipermail/binutils/2020-October/113744.html > > > > is needed for binutils master branch. BTW, binutils 2.35 and 2.35.1 are > > broken. > > > > > compilation no matter whether the compiler passes -gdwarf-2 to gas or not > > > (assuming dwarf 2 is the default in gas) when compiling various *.c/*.C > > > files? Ditto for modified gas that would default to other .debug_line > > > versions and corresponding gcc -gdwarf-N flag and passing vs. not passing > > > that -gdwarf-N to gas? > > > > > > The hope is that -gdwarf-N to gas would have two functions: > > > 1) if the assembly doesn't contain .debug* sections/.file/.loc directives, > > > emit debug info (mainly .debug_line) for the assembler file in order to > > > be able to debug those > > > 2) if the assembly does contain .debug* sections/.file/.loc directives, > > > only change the version of the .debug_line generated for the .file/.loc > > > directives, but nothing else; keep all .debug* sections but .debug_line > > > as is > > > > Here is the updated patch. OK for master? > The updated patch to check https://sourceware.org/bugzilla/show_bug.cgi?id=26778 OK for master? From 7130dea1f39692a8a00e48ef119233595f9284c9 Mon Sep 17 00:00:00 2001 From: "H.J. Lu" Date: Fri, 16 Oct 2020 05:59:51 -0700 Subject: [PATCH] Update check for working assembler --gdwarf-4 option Rename HAVE_AS_WORKING_DWARF_4_FLAG to HAVE_AS_WORKING_DWARF_N_FLAG Don't set HAVE_AS_WORKING_DWARF_N_FLAG if --gdwarf-5/--gdwarf-4 generate an extra assembly input file in debug info from compiler generated .debug_line or fail with the APP marker: https://sourceware.org/bugzilla/show_bug.cgi?id=25878 https://sourceware.org/bugzilla/show_bug.cgi?id=26740 https://sourceware.org/bugzilla/show_bug.cgi?id=26778 Also replace success with dwarf4_success in the 32-bit --gdwarf-4 check. PR bootstrap/97451 * configure.ac (HAVE_AS_WORKING_DWARF_4_FLAG): Renamed to ... (HAVE_AS_WORKING_DWARF_N_FLAG): This. Don't define if there is an extra assembly input file in debug info. Replace success with dwarf4_success in the 32-bit --gdwarf-4 check. * dwarf2out.c (asm_outputs_debug_line_str): Check HAVE_AS_WORKING_DWARF_N_FLAG instead of HAVE_AS_WORKING_DWARF_4_FLAG. * gcc.c (ASM_DEBUG_SPEC): Likewise. (ASM_DEBUG_OPTION_SPEC): Likewise. * config.in: Regenerated. * configure: Likewise. --- gcc/config.in | 6 +-- gcc/configure | 106 +++++++++++++++++++++++++++++++++++++++++++++-- gcc/configure.ac | 59 ++++++++++++++++++++++++-- gcc/dwarf2out.c | 2 +- gcc/gcc.c | 4 +- 5 files changed, 164 insertions(+), 13 deletions(-) diff --git a/gcc/config.in b/gcc/config.in index 3657c46f349..b7c3107bfe3 100644 --- a/gcc/config.in +++ b/gcc/config.in @@ -719,10 +719,10 @@ #endif -/* Define if your assembler supports --gdwarf-4 even with compiler generated - .debug_line */ +/* Define if your assembler supports --gdwarf-4/--gdwarf-5 even with compiler + generated .debug_line. */ #ifndef USED_FOR_TARGET -#undef HAVE_AS_WORKING_DWARF_4_FLAG +#undef HAVE_AS_WORKING_DWARF_N_FLAG #endif diff --git a/gcc/configure b/gcc/configure index abff47d30eb..f96a89e8c37 100755 --- a/gcc/configure +++ b/gcc/configure @@ -28701,7 +28701,7 @@ fi { $as_echo "$as_me:${as_lineno-$LINENO}: result: $gcc_cv_as_debug_line_32_flag" >&5 $as_echo "$gcc_cv_as_debug_line_32_flag" >&6; } if test $gcc_cv_as_debug_line_32_flag = yes; then - success=yes + dwarf4_success=yes fi else @@ -28741,6 +28741,7 @@ fi fi if test $dwarf4_success = yes; then + dwarf4_success=no { $as_echo "$as_me:${as_lineno-$LINENO}: checking assembler for --gdwarf-4 not refusing compiler generated .debug_line" >&5 $as_echo_n "checking assembler for --gdwarf-4 not refusing compiler generated .debug_line... " >&6; } if ${gcc_cv_as_dwarf_4_debug_line_flag+:} false; then : @@ -28772,9 +28773,7 @@ fi { $as_echo "$as_me:${as_lineno-$LINENO}: result: $gcc_cv_as_dwarf_4_debug_line_flag" >&5 $as_echo "$gcc_cv_as_dwarf_4_debug_line_flag" >&6; } if test $gcc_cv_as_dwarf_4_debug_line_flag = yes; then - -$as_echo "#define HAVE_AS_WORKING_DWARF_4_FLAG 1" >>confdefs.h - + dwarf4_success=yes fi break @@ -28785,6 +28784,105 @@ fi dwarf4_line_sz=5 done + if test $dwarf4_success = yes; then + conftest_s="\ + .file \"foo.c\" + .text +bar: +#APP +# 82 \"xxx.h\" 1 + $insn +# 0 \"\" 2 +#NO_APP + $insn +foo: + .file 1 \"foo.c\" + $insn + .file 2 \"foo.h\" + ret +" + dwarf4_success=no + { $as_echo "$as_me:${as_lineno-$LINENO}: checking assembler for --gdwarf-4 with the APP marker" >&5 +$as_echo_n "checking assembler for --gdwarf-4 with the APP marker... " >&6; } +if ${gcc_cv_as_dwarf_4_app_flag+:} false; then : + $as_echo_n "(cached) " >&6 +else + gcc_cv_as_dwarf_4_app_flag=no + if test x$gcc_cv_as != x; then + $as_echo "$conftest_s" > conftest.s + if { ac_try='$gcc_cv_as $gcc_cv_as_flags --gdwarf-4 -o conftest.o conftest.s >&5' + { { eval echo "\"\$as_me\":${as_lineno-$LINENO}: \"$ac_try\""; } >&5 + (eval $ac_try) 2>&5 + ac_status=$? + $as_echo "$as_me:${as_lineno-$LINENO}: \$? = $ac_status" >&5 + test $ac_status = 0; }; } + then + gcc_cv_as_dwarf_4_app_flag=yes + else + echo "configure: failed program was" >&5 + cat conftest.s >&5 + fi + rm -f conftest.o conftest.s + fi +fi +{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $gcc_cv_as_dwarf_4_app_flag" >&5 +$as_echo "$gcc_cv_as_dwarf_4_app_flag" >&6; } +if test $gcc_cv_as_dwarf_4_app_flag = yes; then + dwarf4_success=yes +fi + + fi + + if test $dwarf4_success = yes; then + conftest_s="\ + .text + .globl foo + .type foo, %function +foo: + $insn + .size foo, .-foo + .file 1 \"foo.c\" +" + { $as_echo "$as_me:${as_lineno-$LINENO}: checking assembler for working --gdwarf-4/--gdwarf-5 for all sources" >&5 +$as_echo_n "checking assembler for working --gdwarf-4/--gdwarf-5 for all sources... " >&6; } +if ${gcc_cv_as_working_gdwarf_n_flag+:} false; then : + $as_echo_n "(cached) " >&6 +else + gcc_cv_as_working_gdwarf_n_flag=no + if test x$gcc_cv_as != x; then + $as_echo "$conftest_s" > conftest.s + if { ac_try='$gcc_cv_as $gcc_cv_as_flags --gdwarf-4 -o conftest.o conftest.s >&5' + { { eval echo "\"\$as_me\":${as_lineno-$LINENO}: \"$ac_try\""; } >&5 + (eval $ac_try) 2>&5 + ac_status=$? + $as_echo "$as_me:${as_lineno-$LINENO}: \$? = $ac_status" >&5 + test $ac_status = 0; }; } + then + if test x$gcc_cv_readelf != x \ + && $gcc_cv_readelf -w conftest.o 2>&1 \ + | grep conftest.s > /dev/null 2>&1; then + gcc_cv_as_working_gdwarf_n_flag=no + else + gcc_cv_as_working_gdwarf_n_flag=yes + fi + else + echo "configure: failed program was" >&5 + cat conftest.s >&5 + fi + rm -f conftest.o conftest.s + fi +fi +{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $gcc_cv_as_working_gdwarf_n_flag" >&5 +$as_echo "$gcc_cv_as_working_gdwarf_n_flag" >&6; } + + + if test $gcc_cv_as_working_gdwarf_n_flag = yes; then + +$as_echo "#define HAVE_AS_WORKING_DWARF_N_FLAG 1" >>confdefs.h + + fi + fi + { $as_echo "$as_me:${as_lineno-$LINENO}: checking assembler for --gstabs option" >&5 $as_echo_n "checking assembler for --gstabs option... " >&6; } if ${gcc_cv_as_gstabs_flag+:} false; then : diff --git a/gcc/configure.ac b/gcc/configure.ac index 26a5d8e3619..9154a5475dc 100644 --- a/gcc/configure.ac +++ b/gcc/configure.ac @@ -5364,7 +5364,7 @@ foo: gcc_GAS_CHECK_FEATURE([assembly of compiler generated 32-bit .debug_line], gcc_cv_as_debug_line_32_flag, [elf,2,36,0], [], [$conftest_s],, - [success=yes]) + [dwarf4_success=yes]) else gcc_GAS_CHECK_FEATURE([assembly of compiler generated 64-bit .debug_line], gcc_cv_as_debug_line_64_flag, @@ -5372,11 +5372,11 @@ foo: [dwarf4_success=yes]) fi if test $dwarf4_success = yes; then + dwarf4_success=no gcc_GAS_CHECK_FEATURE([--gdwarf-4 not refusing compiler generated .debug_line], gcc_cv_as_dwarf_4_debug_line_flag, [elf,2,36,0], [--gdwarf-4], [$conftest_s],, - [AC_DEFINE(HAVE_AS_WORKING_DWARF_4_FLAG, 1, -[Define if your assembler supports --gdwarf-4 even with compiler generated .debug_line])]) + [dwarf4_success=yes]) break fi dwarf4_debug_info_size=0x36 @@ -5385,6 +5385,59 @@ foo: dwarf4_line_sz=5 done + if test $dwarf4_success = yes; then + conftest_s="\ + .file \"foo.c\" + .text +bar: +#APP +# 82 \"xxx.h\" 1 + $insn +# 0 \"\" 2 +#NO_APP + $insn +foo: + .file 1 \"foo.c\" + $insn + .file 2 \"foo.h\" + ret +" + dwarf4_success=no + gcc_GAS_CHECK_FEATURE([--gdwarf-4 with the APP marker], + gcc_cv_as_dwarf_4_app_flag,, + [--gdwarf-4], [$conftest_s],, [dwarf4_success=yes]) + fi + + if test $dwarf4_success = yes; then + conftest_s="\ + .text + .globl foo + .type foo, %function +foo: + $insn + .size foo, .-foo + .file 1 \"foo.c\" +" + gcc_GAS_CHECK_FEATURE([working --gdwarf-4/--gdwarf-5 for all sources], + gcc_cv_as_working_gdwarf_n_flag,, + [--gdwarf-4], + [$conftest_s], + [changequote(,)dnl + if test x$gcc_cv_readelf != x \ + && $gcc_cv_readelf -w conftest.o 2>&1 \ + | grep conftest.s > /dev/null 2>&1; then + gcc_cv_as_working_gdwarf_n_flag=no + else + gcc_cv_as_working_gdwarf_n_flag=yes + fi + changequote([,])dnl]) + if test $gcc_cv_as_working_gdwarf_n_flag = yes; then + AC_DEFINE(HAVE_AS_WORKING_DWARF_N_FLAG, 1, +[Define if your assembler supports --gdwarf-4/--gdwarf-5 even with + compiler generated .debug_line.]) + fi + fi + gcc_GAS_CHECK_FEATURE([--gstabs option], gcc_cv_as_gstabs_flag, [elf,2,11,0], [--gstabs], [$insn],, diff --git a/gcc/dwarf2out.c b/gcc/dwarf2out.c index ba93a6c3d81..534877babfb 100644 --- a/gcc/dwarf2out.c +++ b/gcc/dwarf2out.c @@ -11884,7 +11884,7 @@ asm_outputs_debug_line_str (void) return true; else { -#if defined(HAVE_AS_GDWARF_5_DEBUG_FLAG) && defined(HAVE_AS_WORKING_DWARF_4_FLAG) +#if defined(HAVE_AS_GDWARF_5_DEBUG_FLAG) && defined(HAVE_AS_WORKING_DWARF_N_FLAG) return !dwarf_split_debug_info && dwarf_version >= 5; #else return false; diff --git a/gcc/gcc.c b/gcc/gcc.c index 337c27442a3..cdf4d4f2403 100644 --- a/gcc/gcc.c +++ b/gcc/gcc.c @@ -880,7 +880,7 @@ proper position among the other output files. */ /* Define ASM_DEBUG_SPEC to be a spec suitable for translating '-g' to the assembler, when compiling assembly sources only. */ #ifndef ASM_DEBUG_SPEC -# if defined(HAVE_AS_GDWARF_5_DEBUG_FLAG) && defined(HAVE_AS_WORKING_DWARF_4_FLAG) +# if defined(HAVE_AS_GDWARF_5_DEBUG_FLAG) && defined(HAVE_AS_WORKING_DWARF_N_FLAG) /* If --gdwarf-N is supported and as can handle even compiler generated .debug_line with it, supply --gdwarf-N in ASM_DEBUG_OPTION_SPEC rather than in ASM_DEBUG_SPEC, so that it applies to both .s and .c etc. @@ -921,7 +921,7 @@ proper position among the other output files. */ /* Define ASM_DEBUG_OPTION_SPEC to be a spec suitable for translating '-g' to the assembler when compiling all sources. */ #ifndef ASM_DEBUG_OPTION_SPEC -# if defined(HAVE_AS_GDWARF_5_DEBUG_FLAG) && defined(HAVE_AS_WORKING_DWARF_4_FLAG) +# if defined(HAVE_AS_GDWARF_5_DEBUG_FLAG) && defined(HAVE_AS_WORKING_DWARF_N_FLAG) # define ASM_DEBUG_OPTION_DWARF_OPT \ "%{%:dwarf-version-gt(4):--gdwarf-5 ;" \ "%:dwarf-version-gt(3):--gdwarf-4 ;" \ -- 2.26.2