From: Nick Clifton <nickc@redhat.com>
To: "H.J. Lu" <hjl.tools@gmail.com>, binutils@sourceware.org
Subject: Re: [PATCH] ELF: Consolidate readonly_dynrelocs
Date: Wed, 3 Jun 2020 16:54:59 +0100 [thread overview]
Message-ID: <4eca6409-449b-8a8e-6180-6c8ebf29708f@redhat.com> (raw)
In-Reply-To: <20200602120911.283308-1-hjl.tools@gmail.com>
Hi H.J.
> I will check it in if there are no objections.
Go for it.
> PR ld/26067
> * elf-bfd.h (_bfd_elf_readonly_dynrelocs): New.
> * elf32-arm.c (readonly_dynrelocs): Removed.
> (maybe_set_textrel): Replace readonly_dynrelocs with
> _bfd_elf_readonly_dynrelocs.
> * elf32-csky.c (readonly_dynrelocs): Removed.
> (maybe_set_textrel): Replace readonly_dynrelocs with
> _bfd_elf_readonly_dynrelocs.
> * elf32-hppa.c(readonly_dynrelocs): Removed.
> (alias_readonly_dynrelocs): Replace readonly_dynrelocs with
> _bfd_elf_readonly_dynrelocs.
> (maybe_set_textrel): Likewise.
> * elf32-lm32.c (readonly_dynrelocs): Removed.
> (lm32_elf_adjust_dynamic_symbol): Replace readonly_dynrelocs
> with _bfd_elf_readonly_dynrelocs.
> (maybe_set_textrel): Likewise.
> * elf32-m32r.c (readonly_dynrelocs): Removed.
> (m32r_elf_adjust_dynamic_symbol): Replace readonly_dynrelocs
> with _bfd_elf_readonly_dynrelocs.
> (maybe_set_textrel): Likewise.
> * elf32-metag.c (readonly_dynrelocs): Removed.
> (elf_metag_adjust_dynamic_symbol): Replace readonly_dynrelocs
> with _bfd_elf_readonly_dynrelocs.
> (maybe_set_textrel): Likewise.
> * elf32-microblaze.c (readonly_dynrelocs): Removed.
> (microblaze_elf_adjust_dynamic_symbol): Replace readonly_dynrelocs
> with _bfd_elf_readonly_dynrelocs.
> * elf32-nds32.c (readonly_dynrelocs): Removed.
> (nds32_elf_adjust_dynamic_symbol): Replace readonly_dynrelocs
> with _bfd_elf_readonly_dynrelocs.
> (maybe_set_textrel): Likewise.
> * elf32-or1k.c (readonly_dynrelocs): Removed.
> (or1k_elf_adjust_dynamic_symbol): Replace readonly_dynrelocs
> with _bfd_elf_readonly_dynrelocs.
> * elf32-ppc.c (readonly_dynrelocs): Removed.
> (alias_readonly_dynrelocs): Replace readonly_dynrelocs with
> _bfd_elf_readonly_dynrelocs.
> (ppc_elf_adjust_dynamic_symbol): Likewise.
> (maybe_set_textrel): Likewise.
> * elf32-s390.c (readonly_dynrelocs): Removed.
> (elf_s390_adjust_dynamic_symbol): Replace readonly_dynrelocs
> with _bfd_elf_readonly_dynrelocs.
> (maybe_set_textrel): Likewise.
> * elf32-sh.c (readonly_dynrelocs): Removed.
> (sh_elf_adjust_dynamic_symbol): Replace readonly_dynrelocs with
> _bfd_elf_readonly_dynrelocs.
> (maybe_set_textrel): Likewise.
> * elf32-tic6x.c (readonly_dynrelocs): Removed.
> (maybe_set_textrel): Replace readonly_dynrelocs with
> _bfd_elf_readonly_dynrelocs.
> * elf32-tilepro.c (readonly_dynrelocs): Removed.
> (tilepro_elf_adjust_dynamic_symbol): Replace readonly_dynrelocs
> with _bfd_elf_readonly_dynrelocs.
> (maybe_set_textrel): Likewise.
> * elf64-ppc.c (readonly_dynrelocs): Removed.
> (alias_readonly_dynrelocs): Replace readonly_dynrelocs with
> _bfd_elf_readonly_dynrelocs.
> (ppc64_elf_adjust_dynamic_symbol): Likewise.
> (maybe_set_textrel): Likewise.
> * elf64-s390.c (readonly_dynrelocs): Removed.
> (elf_s390_adjust_dynamic_symbol): Replace readonly_dynrelocs
> with _bfd_elf_readonly_dynrelocs.
> (maybe_set_textrel): Likewise.
> * elflink.c (_bfd_elf_readonly_dynrelocs): New.
> * elfnn-aarch64.c (readonly_dynrelocs): Removed.
> (maybe_set_textrel): Replace readonly_dynrelocs with
> _bfd_elf_readonly_dynrelocs.
> * elfnn-riscv.c (readonly_dynrelocs): Removed.
> (riscv_elf_adjust_dynamic_symbol): Replace readonly_dynrelocs
> with _bfd_elf_readonly_dynrelocs.
> (maybe_set_textrel): Likewise.
> * elfxx-sparc.c (readonly_dynrelocs): Removed.
> (_bfd_sparc_elf_adjust_dynamic_symbol): Replace
> readonly_dynrelocs with _bfd_elf_readonly_dynrelocs.
> (maybe_set_textrel): Likewise.
> * elfxx-tilegx.c (readonly_dynrelocs): Removed.
> (tilegx_elf_adjust_dynamic_symbol): Replace readonly_dynrelocs
> with _bfd_elf_readonly_dynrelocs.
> (maybe_set_textrel): Likewise.
> * elfxx-x86.c (readonly_dynrelocs): Removed.
> (maybe_set_textrel): Replace readonly_dynrelocs with
> _bfd_elf_readonly_dynrelocs.
> (_bfd_x86_elf_adjust_dynamic_symbol): Likewise.
Approved - please apply.
Cheers
Nick
prev parent reply other threads:[~2020-06-03 15:55 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-02 12:09 H.J. Lu
2020-06-03 15:54 ` Nick Clifton [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4eca6409-449b-8a8e-6180-6c8ebf29708f@redhat.com \
--to=nickc@redhat.com \
--cc=binutils@sourceware.org \
--cc=hjl.tools@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).