From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from esa1.mentor.iphmx.com (esa1.mentor.iphmx.com [68.232.129.153]) by sourceware.org (Postfix) with ESMTPS id 9BA363858D20 for ; Fri, 4 Nov 2022 09:32:28 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 9BA363858D20 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=codesourcery.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=mentor.com X-IronPort-AV: E=Sophos;i="5.96,137,1665475200"; d="scan'208";a="88962418" Received: from orw-gwy-02-in.mentorg.com ([192.94.38.167]) by esa1.mentor.iphmx.com with ESMTP; 04 Nov 2022 01:32:27 -0800 IronPort-SDR: RDHrNyUjizGyoW8I8ZzH9fCxbOtv+dInVMQcR/ymShK1lLjW3HEX9SL1isBJkq9edHgxKvBXfR 9SiUXYSaU4VzFkGT2WRdrP/wx/Nr80LXIS0XtJCR7aDCVHq3TRXelyxocpRPx/LJNseKKwdVua sekYCi6lXvPqt2ejMh78v9s9jV1cpKoMpUVusWKQ4kRP8COE6SCvvMnD7DHAy7c5XSlgz7DLAV TYtuJb5iwcHWz73bolC4Z8mJqw+qarc0xXmSRKEQ5/TsGfUYxVmEp2p4ghHECGcBg9+ikYt0dM QXA= From: Thomas Schwinge To: Martin =?utf-8?Q?Li=C5=A1ka?= , Richard Biener CC: Subject: Re: [PATCH 1/3] STABS: remove -gstabs and -gxcoff functionality In-Reply-To: <10a94ccc-e01b-b98a-0fcb-cd661c10c315@suse.cz> References: <10a94ccc-e01b-b98a-0fcb-cd661c10c315@suse.cz> User-Agent: Notmuch/0.29.3+94~g74c3f1b (https://notmuchmail.org) Emacs/27.1 (x86_64-pc-linux-gnu) Date: Fri, 4 Nov 2022 10:32:20 +0100 Message-ID: <87k04b2hgb.fsf@euler.schwinge.homeip.net> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-Originating-IP: [137.202.0.90] X-ClientProxiedBy: svr-ies-mbx-09.mgc.mentorg.com (139.181.222.9) To svr-ies-mbx-10.mgc.mentorg.com (139.181.222.10) X-Spam-Status: No, score=-6.3 required=5.0 tests=BAYES_00,HEADER_FROM_DIFFERENT_DOMAINS,KAM_DMARC_STATUS,SPF_HELO_PASS,SPF_PASS,TXREP,UPPERCASE_50_75 autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: Hi! On 2022-09-01T12:05:23+0200, Martin Li=C5=A1ka wrote: > gcc/ChangeLog: > --- a/gcc/system.h > +++ b/gcc/system.h > @@ -1009,8 +1009,7 @@ extern void fancy_abort (const char *, int, const c= har *) > ASM_OUTPUT_DEFINE_LABEL_DIFFERENCE_SYMBOL HOST_WORDS_BIG_ENDIAN = \ > OBJC_PROLOGUE ALLOCATE_TRAMPOLINE HANDLE_PRAGMA ROUND_TYPE_SIZE = \ > ROUND_TYPE_SIZE_UNIT CONST_SECTION_ASM_OP CRT_GET_RFIB_TEXT = \ > - DBX_LBRAC_FIRST DBX_OUTPUT_ENUM DBX_OUTPUT_SOURCE_FILENAME = \ > - DBX_WORKING_DIRECTORY INSN_CACHE_DEPTH INSN_CACHE_SIZE = \ > + INSN_CACHE_DEPTH INSN_CACHE_SIZE = \ > INSN_CACHE_LINE_WIDTH INIT_SECTION_PREAMBLE NEED_ATEXIT ON_EXIT = \ > EXIT_BODY OBJECT_FORMAT_ROSE MULTIBYTE_CHARS MAP_CHARACTER = \ > LIBGCC_NEEDS_DOUBLE FINAL_PRESCAN_LABEL DEFAULT_CALLER_SAVES = \ > @@ -1023,15 +1022,14 @@ extern void fancy_abort (const char *, int, const= char *) > MAX_WCHAR_TYPE_SIZE SHARED_SECTION_ASM_OP INTEGRATE_THRESHOLD = \ > FINAL_REG_PARM_STACK_SPACE MAYBE_REG_PARM_STACK_SPACE = \ > TRADITIONAL_PIPELINE_INTERFACE DFA_PIPELINE_INTERFACE = \ > - DBX_OUTPUT_STANDARD_TYPES BUILTIN_SETJMP_FRAME_VALUE = \ > + BUILTIN_SETJMP_FRAME_VALUE = \ > SUNOS4_SHARED_LIBRARIES PROMOTE_FOR_CALL_ONLY = \ > SPACE_AFTER_L_OPTION NO_RECURSIVE_FUNCTION_CSE = \ > DEFAULT_MAIN_RETURN TARGET_MEM_FUNCTIONS EXPAND_BUILTIN_VA_ARG = \ > COLLECT_PARSE_FLAG DWARF2_GENERATE_TEXT_SECTION_LABEL WINNING_GDB = \ > ASM_OUTPUT_FILENAME ASM_OUTPUT_SOURCE_LINE FILE_NAME_JOINER = \ > - GDB_INV_REF_REGPARM_STABS_LETTER DBX_MEMPARM_STABS_LETTER = \ > - PUT_SDB_SRC_FILE STABS_GCC_MARKER DBX_OUTPUT_FUNCTION_END = \ > - DBX_OUTPUT_GCC_MARKER DBX_FINISH_SYMBOL SDB_GENERATE_FAKE = \ > + GDB_INV_REF_REGPARM_STABS_LETTER = \ > + PUT_SDB_SRC_FILE STABS_GCC_MARKER SDB_GENERATE_FAKE = \ > NON_SAVING_SETJMP TARGET_LATE_RTL_PROLOGUE_EPILOGUE = \ > CASE_DROPS_THROUGH TARGET_BELL TARGET_BS TARGET_CR TARGET_DIGIT0 = \ > TARGET_ESC TARGET_FF TARGET_NEWLINE TARGET_TAB TARGET_VT = \ > @@ -1056,8 +1054,8 @@ extern void fancy_abort (const char *, int, const c= har *) > PREFERRED_OUTPUT_RELOAD_CLASS SYSTEM_INCLUDE_DIR = \ > STANDARD_INCLUDE_DIR STANDARD_INCLUDE_COMPONENT = \ > LINK_ELIMINATE_DUPLICATE_LDIRECTORIES MIPS_DEBUGGING_INFO = \ > - IDENT_ASM_OP ALL_COP_ADDITIONAL_REGISTER_NAMES DBX_OUTPUT_LBRAC = \ > - DBX_OUTPUT_NFUN DBX_OUTPUT_RBRAC RANGE_TEST_NON_SHORT_CIRCUIT = \ > + IDENT_ASM_OP ALL_COP_ADDITIONAL_REGISTER_NAMES = \ > + RANGE_TEST_NON_SHORT_CIRCUIT = \ > REAL_VALUE_TRUNCATE REVERSE_CONDEXEC_PREDICATES_P = \ > TARGET_ALIGN_ANON_BITFIELDS TARGET_NARROW_VOLATILE_BITFIELDS = \ > IDENT_ASM_OP UNALIGNED_SHORT_ASM_OP UNALIGNED_INT_ASM_OP = \ These changes are part of commit r13-2361-g7e0db0cdf01e9c885a29cb37415f5bc00d90c029 "STABS: remove -gstabs and -gxcoff functionality". What this does is remove these identifiers from "poisoning": /* As the last action in this file, we poison the identifiers that shouldn't be used. [...] /* Other obsolete target macros, or macros that used to be in target headers and were not used, and may be obsolete or may never have been used. */ #pragma GCC poison [...] Shouldn't these identifiers actually stay (so that any accidental future use gets flagged, as I understand this machinery), and instead more identifiers be added potentially: those where their definition/use got removed with "STABS: remove -gstabs and -gxcoff functionality"? (I've not checked.) Gr=C3=BC=C3=9Fe Thomas ----------------- Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstra=C3=9Fe 201= , 80634 M=C3=BCnchen; Gesellschaft mit beschr=C3=A4nkter Haftung; Gesch=C3= =A4ftsf=C3=BChrer: Thomas Heurung, Frank Th=C3=BCrauf; Sitz der Gesellschaf= t: M=C3=BCnchen; Registergericht M=C3=BCnchen, HRB 106955