[binutils-gdb] x86: don't needlessly override .bss

Jan Beulich jbeulich@sourceware.org
Fri Dec 15 11:42:57 GMT 2023


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=d4064aad87f47dac73f9ae2b2c24bae8bb5feabe

commit d4064aad87f47dac73f9ae2b2c24bae8bb5feabe
Author: Jan Beulich <jbeulich@suse.com>
Date:   Fri Dec 15 12:40:52 2023 +0100

    x86: don't needlessly override .bss
    
    ELF, COFF, and Mach-O all have custom handlers for .bss. Don't override
    those; install a handler only for a.out.

Diff:
---
 gas/config/tc-i386.c | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/gas/config/tc-i386.c b/gas/config/tc-i386.c
index f1473240ac6..6a349f2ccc5 100644
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -181,7 +181,7 @@ static const reg_entry *build_modrm_byte (void);
 static void output_insn (const struct last_insn *);
 static void output_imm (fragS *, offsetT);
 static void output_disp (fragS *, offsetT);
-#ifndef I386COFF
+#ifdef OBJ_AOUT
 static void s_bss (int);
 #endif
 #if defined (OBJ_ELF) || defined (OBJ_MAYBE_ELF)
@@ -1201,9 +1201,10 @@ const pseudo_typeS md_pseudo_table[] =
   {"align", s_align_ptwo, 0},
 #endif
   {"arch", set_cpu_arch, 0},
-#ifndef I386COFF
+#ifdef OBJ_AOUT
   {"bss", s_bss, 0},
-#else
+#endif
+#ifdef I386COFF
   {"lcomm", pe_lcomm, 1},
 #endif
   {"ffloat", float_cons, 'f'},
@@ -15477,17 +15478,13 @@ md_pcrel_from (fixS *fixP)
   return fixP->fx_size + fixP->fx_where + fixP->fx_frag->fr_address;
 }
 
-#ifndef I386COFF
+#ifdef OBJ_AOUT
 
 static void
 s_bss (int ignore ATTRIBUTE_UNUSED)
 {
   int temp;
 
-#if defined (OBJ_ELF) || defined (OBJ_MAYBE_ELF)
-  if (IS_ELF)
-    obj_elf_section_change_hook ();
-#endif
   temp = get_absolute_expression ();
   subseg_set (bss_section, (subsegT) temp);
   demand_empty_rest_of_line ();


More information about the Binutils-cvs mailing list