* [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).