From ef744040b49e92ecd3de3ae3b94c8f85020111c3 Mon Sep 17 00:00:00 2001 From: Tucker Date: Mon, 12 Jul 2021 17:12:13 +0100 Subject: [PATCH] Add the SEC_ELF_OCTETS flag to debug sections created by the assembler. PR 28054 gas * config/obj-elf.c (obj_elf_change_section): Set the SEF_ELF_OCTETS flag on debug sections. --- gas/ChangeLog | 6 ++++++ gas/config/obj-elf.c | 16 ++++++++++++++++ 2 files changed, 22 insertions(+) diff --git a/gas/ChangeLog b/gas/ChangeLog index 0bcf9ffb93a..432506baf57 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,3 +1,9 @@ +2021-07-12 Tucker + + PR 28054 + * config/obj-elf.c (obj_elf_change_section): Set the + SEF_ELF_OCTETS flag on debug sections. + 2021-06-07 Andreas Krebbel * testsuite/gas/s390/zarch-arch14.d: Add qpaci. diff --git a/gas/config/obj-elf.c b/gas/config/obj-elf.c index e93d59cb081..42a3851e772 100644 --- a/gas/config/obj-elf.c +++ b/gas/config/obj-elf.c @@ -743,6 +743,22 @@ obj_elf_change_section (const char *name, if (linkonce) flags |= SEC_LINK_ONCE | SEC_LINK_DUPLICATES_DISCARD; + /* PR 28054: Set the SEC_ELF_OCTETS flag for debugging sections. + Based on the code in bfd/elf.c:_bfd_elf_make_section_from_shdr(). + + FIXME: We do not set the SEC_DEBUGGING flag because that causes + problems for the FT32 and MSP430 targets. Investigate and fix. */ + if ((flags & SEC_ALLOC) == 0 && name [0] == '.') + { + if ( startswith (name, ".debug") + || startswith (name, ".zdebug") + || startswith (name, ".gnu.debuglto_.debug_") + || startswith (name, ".gnu.linkonce.wi.") + || startswith (name, GNU_BUILD_ATTRS_SECTION_NAME) + || startswith (name, ".note.gnu")) + flags |= SEC_ELF_OCTETS; + } + if (old_sec == NULL) { symbolS *secsym; -- 2.30.2