[binutils-gdb] bfd: riscv_maybe_function_sym check _bfd_elf_is_local_label_name

Mark Wielaard mark@sourceware.org
Mon Jan 15 13:05:30 GMT 2024


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

commit 907aee5baf05bf54faa950aba52ed09fc79085eb
Author: Mark Wielaard <mark@klomp.org>
Date:   Thu Jan 4 01:31:31 2024 +0100

    bfd: riscv_maybe_function_sym check _bfd_elf_is_local_label_name
    
    This fixes the ld "Handle no DWARF information" testcase. Which
    currently fails on riscv because a local label name is assumed
    to be the current function name.
    
    bfd/ChangeLog:
    
    * elfnn-riscv.c (riscv_maybe_function_sym): Also check
            _bfd_elf_is_local_label_name.

Diff:
---
 bfd/elfnn-riscv.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/bfd/elfnn-riscv.c b/bfd/elfnn-riscv.c
index fa03d1a53ec..8b27e3b8d6a 100644
--- a/bfd/elfnn-riscv.c
+++ b/bfd/elfnn-riscv.c
@@ -5452,7 +5452,8 @@ riscv_maybe_function_sym (const asymbol *sym,
 			  bfd_vma *code_off)
 {
   if (sym->flags & BSF_LOCAL
-      && riscv_elf_is_mapping_symbols (sym->name))
+      && (riscv_elf_is_mapping_symbols (sym->name)
+	  || _bfd_elf_is_local_label_name (sec->owner, sym->name)))
     return 0;
 
   return _bfd_elf_maybe_function_sym (sym, sec, code_off);


More information about the Binutils-cvs mailing list