From: Jan Beulich <jbeulich@suse.com>
Date: Fri, 5 Jan 2024 07:10:01 +0000 (+0100)
Subject: s390: drop .bss override
X-Git-Tag: upstream/2.42~237
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f04eabd6b3a6e514ab644f61ffa6eb10ff319e09;p=platform%2Fupstream%2Fbinutils.git

s390: drop .bss override

The comment looks bogus (perhaps simply stale), and there are also no
other precautions against subsections being used on ELF with .bss. It
also doesn't look to be a good idea to override the custom handler that
ELF has (afaict doing so further broke .previous).
---

diff --git a/gas/config/tc-s390.c b/gas/config/tc-s390.c
index 70d4bdf..09a903a 100644
--- a/gas/config/tc-s390.c
+++ b/gas/config/tc-s390.c
@@ -88,7 +88,6 @@ int s390_cie_data_alignment;
 /* Define the prototypes for the pseudo-ops */
 static void s390_byte (int);
 static void s390_elf_cons (int);
-static void s390_bss (int);
 static void s390_insn (int);
 static void s390_literals (int);
 static void s390_machine (int);
@@ -98,7 +97,6 @@ const pseudo_typeS md_pseudo_table[] =
 {
   { "align",        s_align_bytes,      0 },
   /* Pseudo-ops which must be defined.  */
-  { "bss",          s390_bss,           0 },
   { "insn",         s390_insn,          0 },
   /* Pseudo-ops which must be overridden.  */
   { "byte",	    s390_byte,	        0 },
@@ -1734,16 +1732,6 @@ md_create_long_jump (ptr, from_addr, to_addr, frag, to_symbol)
 }
 #endif
 
-void
-s390_bss (int ignore ATTRIBUTE_UNUSED)
-{
-  /* We don't support putting frags in the BSS segment, we fake it
-     by marking in_bss, then looking at s_skip for clues.  */
-
-  subseg_set (bss_section, 0);
-  demand_empty_rest_of_line ();
-}
-
 /* Pseudo-op handling.  */
 
 void