]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
Arm64: drop .bss override
authorJan Beulich <jbeulich@suse.com>
Fri, 5 Jan 2024 07:04:42 +0000 (08:04 +0100)
committerJan Beulich <jbeulich@suse.com>
Fri, 5 Jan 2024 07:04:42 +0000 (08:04 +0100)
The comment looks bogus (perhaps simply stale, perhaps wrongly copied
from Arm in the first place), 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).

As to the mapping state update - such also doesn't appear to be done
for other section switching, so its original purpose was at best
questionable as well.

gas/config/tc-aarch64.c
gas/doc/c-aarch64.texi

index 11707fd2ef8948938b9cae4819fe5759a0f8c142..cf586267ee8592baca17980ec015a96a35427426 100644 (file)
@@ -1866,17 +1866,7 @@ mapping_state_2 (enum mstate state, int max_chars)
 #define mapping_state_2(x, y)  /* nothing */
 #endif
 
-/* 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 ();
-  mapping_state (MAP_DATA);
-}
+/* Directives: alignment.  */
 
 static void
 s_even (int ignore ATTRIBUTE_UNUSED)
@@ -2396,7 +2386,6 @@ const pseudo_typeS md_pseudo_table[] = {
   /* Never called because '.req' does not start a line.  */
   {"req", s_req, 0},
   {"unreq", s_unreq, 0},
-  {"bss", s_bss, 0},
   {"even", s_even, 0},
   {"ltorg", s_ltorg, 0},
   {"pool", s_ltorg, 0},
index ef0f0bccba76e21b7f85afb9c64ff831aba6cfcc..97233b5c0e5248032b18de7dbbbb67ca1d0671dd 100644 (file)
@@ -394,11 +394,6 @@ extensions by the @option{-mcpu} command-line option.
 incrementally to the architecture being compiled for.
 
 @c BBBBBBBBBBBBBBBBBBBBBBBBBB
-
-@cindex @code{.bss} directive, AArch64
-@item .bss
-This directive switches to the @code{.bss} section.
-
 @c CCCCCCCCCCCCCCCCCCCCCCCCCC
 
 @cindex @code{.cpu} directive, AArch64