public inbox for binutils-cvs@sourceware.org
 help / color / mirror / Atom feed
* [binutils-gdb] Arm: drop .bss override
@ 2024-01-05  7:07 Jan Beulich
  0 siblings, 0 replies; only message in thread
From: Jan Beulich @ 2024-01-05  7:07 UTC (permalink / raw)
  To: bfd-cvs

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

commit c621dd904ff6378bb1e97911b5b526bf0a80e345
Author: Jan Beulich <jbeulich@suse.com>
Date:   Fri Jan 5 08:04:18 2024 +0100

    Arm: 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 handlers that
    ELF and COFF have (afaict doing so further broke .previous on ELF).

Diff:
---
 gas/config/tc-arm.c | 16 +---------------
 gas/doc/c-arm.texi  |  5 -----
 2 files changed, 1 insertion(+), 20 deletions(-)

diff --git a/gas/config/tc-arm.c b/gas/config/tc-arm.c
index 74fd50cb9b3..228e5e07eba 100644
--- a/gas/config/tc-arm.c
+++ b/gas/config/tc-arm.c
@@ -3347,20 +3347,7 @@ s_syntax (int unused ATTRIBUTE_UNUSED)
   demand_empty_rest_of_line ();
 }
 
-/* Directives: sectioning and alignment.  */
-
-static void
-s_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 ();
-
-#ifdef md_elf_section_change_hook
-  md_elf_section_change_hook ();
-#endif
-}
+/* Directives: alignment.  */
 
 static void
 s_even (int ignore ATTRIBUTE_UNUSED)
@@ -5187,7 +5174,6 @@ const pseudo_typeS md_pseudo_table[] =
   { "dn",	   s_dn,          0 },
   { "qn",          s_qn,          0 },
   { "unreq",	   s_unreq,	  0 },
-  { "bss",	   s_bss,	  0 },
   { "align",	   s_align_ptwo,  2 },
   { "arm",	   s_arm,	  0 },
   { "thumb",	   s_thumb,	  0 },
diff --git a/gas/doc/c-arm.texi b/gas/doc/c-arm.texi
index de712eddb64..5a951a88220 100644
--- a/gas/doc/c-arm.texi
+++ b/gas/doc/c-arm.texi
@@ -892,11 +892,6 @@ incrementally to the architecture being compiled for.
 This performs the same action as @var{.code 32}.
 
 @c BBBBBBBBBBBBBBBBBBBBBBBBBB
-
-@cindex @code{.bss} directive, ARM
-@item .bss
-This directive switches to the @code{.bss} section.
-
 @c CCCCCCCCCCCCCCCCCCCCCCCCCC
 
 @cindex @code{.cantunwind} directive, ARM

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2024-01-05  7:07 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-01-05  7:07 [binutils-gdb] Arm: drop .bss override Jan Beulich

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).