On 1/2/24 12:41, Ilya Leoshkevich wrote:
gccint recommends using ASM_OUTPUT_FUNCTION_LABEL in
ASM_DECLARE_FUNCTION_NAME, but many implementations use
ASM_OUTPUT_LABEL instead.  It's inconsistent and prevents changes to
ASM_OUTPUT_FUNCTION_LABEL from affecting the respective targets.
---
  gcc/config/aarch64/aarch64.cc       |  2 +-
  gcc/config/alpha/alpha.cc           |  5 ++---
  gcc/config/arm/aout.h               |  2 +-
  gcc/config/arm/arm.cc               |  2 +-
  gcc/config/bfin/bfin.h              | 16 ++++++++--------
  gcc/config/c6x/c6x.h                |  2 +-
  gcc/config/gcn/gcn.cc               |  5 ++---
  gcc/config/h8300/h8300.h            |  2 +-
  gcc/config/ia64/ia64.cc             |  5 ++---
  gcc/config/mcore/mcore-elf.h        |  2 +-
  gcc/config/microblaze/microblaze.cc |  3 +--
  gcc/config/mips/mips.cc             | 19 ++++++++++---------
  gcc/config/pa/pa.cc                 |  3 ++-
  gcc/config/riscv/riscv.cc           |  2 +-
  gcc/config/rs6000/rs6000.cc         |  4 ++--
  15 files changed, 36 insertions(+), 38 deletions(-)
Essentially none of these ports define ASM_OUTPUT_FUCNTION_LABEL and the fallback just uses ASM_OUTPUT_LABEL under the hood. So this should have no functional change. OK for the trunk.

jeff

Reply via email to