[binutils-gdb] visium: drop .bss and .skip overrides

[binutils-gdb] visium: drop .bss and .skip overrides

[Jan Beulich]

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

commit 1ca08a72955dba234586f3cc236be1b343da01b2
Author: Jan Beulich <jbeulich@suse.com>
Date:   Fri Jan 5 08:11:43 2024 +0100

    visium: drop .bss and .skip overrides
    
    The comment in s_bss() looks bogus (perhaps simply stale, or wrongly
    copied from another target). It also doesn't look to be a good idea to
    override the custom handler that ELF has (afaict doing so broke
    .previous as well as sub-section specification).
    
    The override for .skip is simply pointless, for read.c having exactly
    the same.
    
    While there also drop two adjacent redundant (with read.h) declarations
    (which would be outright dangerous if read.h wasn't included anyway).

Diff:
---
 gas/config/tc-visium.c | 23 +----------------------
 1 file changed, 1 insertion(+), 22 deletions(-)

diff --git a/gas/config/tc-visium.c b/gas/config/tc-visium.c
index 8c778427b36..ee20be405da 100644
--- a/gas/config/tc-visium.c
+++ b/gas/config/tc-visium.c
@@ -163,40 +163,19 @@ tc_gen_reloc (asection *section ATTRIBUTE_UNUSED, fixS *fixp)
   return reloc;
 }
 
-extern char *input_line_pointer;
-
-
-static void s_bss (int);
 static void visium_rdata (int);
 
 static void visium_update_parity_bit (char *);
 static char *parse_exp (char *, expressionS *);
 
-/* These are the back-ends for the various machine dependent pseudo-ops.  */
-void demand_empty_rest_of_line (void);
-
-
-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 ();
-}
-
-
 /* This table describes all the machine specific pseudo-ops the assembler
-   has to support. The fields are:
+   has to support, and that aren't handled elsewhere. The fields are:
 
    1: Pseudo-op name without dot.
    2: Function to call to execute this pseudo-op.
    3: Integer arg to pass to the function.  */
 const pseudo_typeS md_pseudo_table[] =
 {
-  {"bss", s_bss, 0},
-  {"skip", s_space, 0},
   {"align", s_align_bytes, 0},
   {"noopt", s_ignore, 0},
   {"optim", s_ignore, 0},