From: Richard Earnshaw <Richard.Earnshaw@foss.arm.com>
To: Frederic Cambus <fred@statdns.com>, binutils@sourceware.org
Subject: Re: [PATCH] Add OpenBSD ARM Little Endian BFD support.
Date: Thu, 1 Sep 2022 10:43:25 +0100 [thread overview]
Message-ID: <b4365edb-a287-02bd-40b5-b2550b3bccb7@foss.arm.com> (raw)
In-Reply-To: <Yw/cyfAKNSuwctZD@Freds-MacBook-Air.local>
On 31/08/2022 23:12, Frederic Cambus wrote:
> On Tue, Aug 30, 2022 at 02:55:00PM +0100, Richard Earnshaw wrote:
>
>> On 27/08/2022 18:28, Frederic Cambus wrote:
>
>>> + arm-*-openbsd*)
>>> + targ_defvec=arm_elf32_le_vec
>>> + targ_selvecs="arm_elf32_be_vec"
>>> + ;;
>>
>> This is exactly the same content as the entry directly above it (netbsd), so
>> please merge the two. It's a maintenance burden to have a lot of duplicated
>> entries.
>
> Makes sense, and note taken. Revised diff inline below, and full patch
> attached for convenience.
Thanks, this is OK.
R.
>
> diff --git a/bfd/config.bfd b/bfd/config.bfd
> index 8c7be5da38a..a79a0e2bf5b 100644
> --- a/bfd/config.bfd
> +++ b/bfd/config.bfd
> @@ -90,7 +90,6 @@ case $targ in
> arm*-*-aout | \
> arm-*-coff | \
> arm*-*-netbsdaout* | \
> - arm-*-openbsd* | \
> arm-*-oabi | \
> arm-*-riscix* | \
> arm*-*-symbianelf* | \
> @@ -383,7 +382,7 @@ case "${targ}" in
> targ_defvec=arm_elf32_be_vec
> targ_selvecs="arm_elf32_le_vec"
> ;;
> - arm-*-netbsd*)
> + arm-*-netbsd* | arm-*-openbsd*)
> targ_defvec=arm_elf32_le_vec
> targ_selvecs="arm_elf32_be_vec"
> ;;
>
next prev parent reply other threads:[~2022-09-01 9:43 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-27 17:28 Frederic Cambus
2022-08-30 13:55 ` Richard Earnshaw
2022-08-31 22:12 ` Frederic Cambus
2022-09-01 9:43 ` Richard Earnshaw [this message]
2022-09-01 15:56 ` Frederic Cambus
2022-09-02 10:31 ` Nick Clifton
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=b4365edb-a287-02bd-40b5-b2550b3bccb7@foss.arm.com \
--to=richard.earnshaw@foss.arm.com \
--cc=binutils@sourceware.org \
--cc=fred@statdns.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).