[binutils-gdb] gas: drop unnecessary use of tc_comment_chars

Jan Beulich jbeulich@sourceware.org
Fri Aug 2 07:45:08 GMT 2024


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

commit 4a9843c301e5453ae21639e500b9b2bd62ff1e69
Author: Jan Beulich <jbeulich@suse.com>
Date:   Fri Aug 2 09:44:53 2024 +0200

    gas: drop unnecessary use of tc_comment_chars
    
    The override is necessary only when a target needs other than an array
    of const char.
    
    For cris drop redundant sibling declarations at the same time.

Diff:
---
 gas/config/tc-cris.c   | 2 +-
 gas/config/tc-cris.h   | 5 -----
 gas/config/tc-mmix.c   | 2 +-
 gas/config/tc-mmix.h   | 3 ---
 gas/config/tc-or1k.c   | 4 +---
 gas/config/tc-or1k.h   | 3 ---
 gas/config/tc-visium.c | 5 ++---
 gas/config/tc-visium.h | 3 ---
 8 files changed, 5 insertions(+), 22 deletions(-)

diff --git a/gas/config/tc-cris.c b/gas/config/tc-cris.c
index 6a916fdd9c7..86ec1f701ec 100644
--- a/gas/config/tc-cris.c
+++ b/gas/config/tc-cris.c
@@ -211,7 +211,7 @@ static int warn_for_branch_expansion = 0;
 static int err_for_dangerous_mul_placement
  = (XCONCAT2 (arch_,DEFAULT_CRIS_ARCH) != arch_crisv32);
 
-const char cris_comment_chars[] = ";";
+const char comment_chars[] = ";";
 
 /* This array holds the chars that only start a comment at the beginning of
    a line.  If the line seems to have the form '# 123 filename'
diff --git a/gas/config/tc-cris.h b/gas/config/tc-cris.h
index 398f8193e86..4dab3adba36 100644
--- a/gas/config/tc-cris.h
+++ b/gas/config/tc-cris.h
@@ -50,11 +50,6 @@ extern size_t md_longopts_size;
 
 extern const pseudo_typeS md_pseudo_table[];
 
-#define tc_comment_chars cris_comment_chars
-extern const char cris_comment_chars[];
-extern const char line_comment_chars[];
-extern const char line_separator_chars[];
-
 /* This should be optional, since it is ignored as an escape (assumed to
    be itself) if it is not recognized.  */
 #define ONLY_STANDARD_ESCAPES
diff --git a/gas/config/tc-mmix.c b/gas/config/tc-mmix.c
index 2ae98b34b4e..4d0f94eda48 100644
--- a/gas/config/tc-mmix.c
+++ b/gas/config/tc-mmix.c
@@ -382,7 +382,7 @@ const pseudo_typeS md_pseudo_table[] =
    {NULL, 0, 0}
  };
 
-const char mmix_comment_chars[] = "%!";
+const char comment_chars[] = "%!";
 
 /* A ':' is a valid symbol character in mmixal.  It's the prefix
    delimiter, but other than that, it works like a symbol character,
diff --git a/gas/config/tc-mmix.h b/gas/config/tc-mmix.h
index b07f0047539..08061b8c123 100644
--- a/gas/config/tc-mmix.h
+++ b/gas/config/tc-mmix.h
@@ -26,9 +26,6 @@
 #define TARGET_ARCH bfd_arch_mmix
 #define TARGET_BYTES_BIG_ENDIAN 1
 
-extern const char mmix_comment_chars[];
-#define tc_comment_chars mmix_comment_chars
-
 extern const char mmix_symbol_chars[];
 #define tc_symbol_chars mmix_symbol_chars
 
diff --git a/gas/config/tc-or1k.c b/gas/config/tc-or1k.c
index 0dcb6ea7d56..14b09c9c3e6 100644
--- a/gas/config/tc-or1k.c
+++ b/gas/config/tc-or1k.c
@@ -49,7 +49,7 @@ typedef struct
 }
 or1k_insn;
 
-const char comment_chars[]        = "#";
+const char comment_chars[]        = ";#";
 const char line_comment_chars[]   = "#";
 const char line_separator_chars[] = ";";
 const char EXP_CHARS[]            = "eE";
@@ -90,8 +90,6 @@ s_nodelay (int val ATTRIBUTE_UNUSED)
   nodelay = true;
 }
 
-const char or1k_comment_chars [] = ";#";
-
 /* The target specific pseudo-ops which we support.  */
 const pseudo_typeS md_pseudo_table[] =
 {
diff --git a/gas/config/tc-or1k.h b/gas/config/tc-or1k.h
index 8277272af3a..5b323f2e6d8 100644
--- a/gas/config/tc-or1k.h
+++ b/gas/config/tc-or1k.h
@@ -29,9 +29,6 @@ extern unsigned long or1k_machine;
 #define TARGET_FORMAT           "elf32-or1k"
 #define TARGET_BYTES_BIG_ENDIAN 1
 
-extern const char or1k_comment_chars [];
-#define tc_comment_chars or1k_comment_chars
-
 /* Permit temporary numeric labels.  */
 #define LOCAL_LABELS_FB 1
 
diff --git a/gas/config/tc-visium.c b/gas/config/tc-visium.c
index ee20be405da..fcb128a1abb 100644
--- a/gas/config/tc-visium.c
+++ b/gas/config/tc-visium.c
@@ -53,9 +53,8 @@
 
 
 /* This string holds the chars that always start a comment. If the
-   pre-processor is disabled, these aren't very useful. The macro
-   tc_comment_chars points to this.  */
-const char *visium_comment_chars = "!;";
+   pre-processor is disabled, these aren't very useful.  */
+const char comment_chars[] = "!;";
 
 /* This array holds the chars that only start a comment at the beginning
    of a line.  If the line seems to have the form '# 123 filename' .line
diff --git a/gas/config/tc-visium.h b/gas/config/tc-visium.h
index c94f27e705d..b85357a3051 100644
--- a/gas/config/tc-visium.h
+++ b/gas/config/tc-visium.h
@@ -63,9 +63,6 @@ extern long visium_pcrel_from_section (struct fix *, segT);
 
 #define md_operand(x)
 
-#define tc_comment_chars visium_comment_chars
-extern const char *visium_comment_chars;
-
 #define TARGET_USE_CFIPOP 1
 
 #define tc_cfi_frame_initial_instructions visium_cfi_frame_initial_instructions


More information about the Binutils-cvs mailing list