[gcc(refs/vendors/ARM/heads/arm-struct-reorg-wip)] RISC-V: Make unique SECCAT_SRODATA names start with .srodata (not .sdata2)

Tamar Christina tnfchris@gcc.gnu.org
Fri Jul 17 15:26:06 GMT 2020


https://gcc.gnu.org/g:18c7b39fdf342e41934467b6c9e4904c1dc7a778

commit 18c7b39fdf342e41934467b6c9e4904c1dc7a778
Author: Keith Packard <keithp@keithp.com>
Date:   Wed Apr 29 09:49:56 2020 -0700

    RISC-V: Make unique SECCAT_SRODATA names start with .srodata (not .sdata2)
    
    default_unique_section uses ".sdata2" as a prefix for SECCAT_SRODATA
    unique sections, but RISC-V uses ".srodata" instead. Override the
    TARGET_ASM_UNIQUE_SECTION function to catch this case, allowing the
    default to be used for all other sections.
    
    Signed-off-by: Keith Packard <keithp@keithp.com>
    
            gcc/
            * config/riscv/riscv.c (riscv_unique_section): New.
            (TARGET_ASM_UNIQUE_SECTION): New.
    
    (cherry picked from commit adce62f53d8ad00e8110a6a2de7962d7a850de16)

Diff:
---
 gcc/config/riscv/riscv.c | 40 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 40 insertions(+)

diff --git a/gcc/config/riscv/riscv.c b/gcc/config/riscv/riscv.c
index 94b5ac01762..43eb7b033b3 100644
--- a/gcc/config/riscv/riscv.c
+++ b/gcc/config/riscv/riscv.c
@@ -3415,6 +3415,43 @@ riscv_select_section (tree decl, int reloc,
     }
 }
 
+/* Switch to the appropriate section for output of DECL.  */
+
+static void
+riscv_unique_section (tree decl, int reloc)
+{
+  const char *prefix = NULL;
+  bool one_only = DECL_ONE_ONLY (decl) && !HAVE_COMDAT_GROUP;
+
+  switch (categorize_decl_for_section (decl, reloc))
+    {
+    case SECCAT_SRODATA:
+      prefix = one_only ? ".sr" : ".srodata";
+      break;
+
+    default:
+      break;
+    }
+  if (prefix)
+    {
+      const char *name, *linkonce;
+      char *string;
+
+      name = IDENTIFIER_POINTER (DECL_ASSEMBLER_NAME (decl));
+      name = targetm.strip_name_encoding (name);
+
+      /* If we're using one_only, then there needs to be a .gnu.linkonce
+	 prefix to the section name.  */
+      linkonce = one_only ? ".gnu.linkonce" : "";
+
+      string = ACONCAT ((linkonce, prefix, ".", name, NULL));
+
+      set_decl_section_name (decl, string);
+      return;
+    }
+  default_unique_section (decl, reloc);
+}
+
 /* Return a section for X, handling small data. */
 
 static section *
@@ -5163,6 +5200,9 @@ riscv_hard_regno_rename_ok (unsigned from_regno ATTRIBUTE_UNUSED,
 #undef TARGET_ASM_SELECT_SECTION
 #define TARGET_ASM_SELECT_SECTION riscv_select_section
 
+#undef TARGET_ASM_UNIQUE_SECTION
+#define TARGET_ASM_UNIQUE_SECTION riscv_unique_section
+
 #undef TARGET_ASM_SELECT_RTX_SECTION
 #define TARGET_ASM_SELECT_RTX_SECTION  riscv_elf_select_rtx_section


More information about the Gcc-cvs mailing list