public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: Mark Harmstone <mark@harmstone.com>
To: "Martin Storsjö" <martin@martin.st>,
	"Tamar Christina" <Tamar.Christina@arm.com>
Cc: Andrew Pinski <pinskia@gmail.com>,
	Richard Earnshaw <Richard.Earnshaw@foss.arm.com>,
	NightStrike <nightstrike@gmail.com>,
	"wej22007@outlook.com" <wej22007@outlook.com>,
	"zac.walker@linaro.org" <zac.walker@linaro.org>,
	binutils <binutils@sourceware.org>,
	"nickc@redhat.com" <nickc@redhat.com>
Subject: Re: [PATCH 1/8] ld: Rename aarch64pe emulation target to arm64pe
Date: Wed, 4 Jan 2023 02:38:54 +0000	[thread overview]
Message-ID: <367317ba-108e-fde8-98d2-0be5146f28fa@harmstone.com> (raw)
In-Reply-To: <237bf7b4-576-945a-58dc-245432e2d9@martin.st>

On 3/1/23 20:05, Martin Storsjö wrote:
 > On Tue, 3 Jan 2023, Tamar Christina wrote:
 >
 >> I don't think we'll need to patch LLVM as you typically don't specify the emulation when using ld.
 >
 > When cross compiling, then the compiler driver (gcc or clang) will specify it.

I'm not 100%, but I'm fairly sure that GCC *always* passes the emulation name to ld.

 > On Tue, 3 Jan 2023, Tamar Christina wrote:
 >
 > The reason for insisting on this change is non-technical in nature. For a project such as binutils we always want the primary name to reflect what the architecture calls things.

Which, while valid, is obviously not important enough to warrant adding unneeded complexity. And ARM Ltd. has at the very least acquiesced in the naming of the architecture as ARM64: see https://developer.arm.com/documentation/102474/0100/Fundamentals-of-Armv8-Neon-technology, "GNU and Linux documentation sometimes refers to AArch64 as ARM64". Or Microsoft's "PE Format" page, which is an official part of the UEFI specification, which uses "ARM64" throughout. And that's before we get to the fact that the corresponding "aarch32" seems to be used hardly anywhere.

It's also worth pointing out that there's no concept of emulation aliases at the moment, so someone would have to develop and test that. And having 300+ emulations, and an alias for just one, seems a little perverse.

Nick, what do you want me to do here?

Mark

  reply	other threads:[~2023-01-04  2:38 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-30  2:40 Mark Harmstone
2022-12-30  2:40 ` [PATCH 2/8] Fix size of external_reloc for pe-aarch64 Mark Harmstone
2022-12-30  2:40 ` [PATCH 3/8] Skip ELF-specific tests when targeting pe-aarch64 Mark Harmstone
2022-12-30  2:40 ` [PATCH 4/8] Skip big-obj test for pe-aarch64 Mark Harmstone
2022-12-30  2:40 ` [PATCH 5/8] Add pe-aarch64 relocations Mark Harmstone
2022-12-30  2:40 ` [PATCH 6/8] Add .secrel32 for pe-aarch64 Mark Harmstone
2022-12-30  2:40 ` [PATCH 7/8] Add aarch64-w64-mingw32 target Mark Harmstone
2022-12-30  2:40 ` [PATCH 8/8] gas: Restore tc_pe_dwarf2_emit_offset for pe-aarch64 Mark Harmstone
2023-01-03 11:54 ` [PATCH 1/8] ld: Rename aarch64pe emulation target to arm64pe Nick Clifton
2023-01-03 11:59 ` NightStrike
2023-01-03 12:09   ` Tamar Christina
2023-01-03 14:08     ` Richard Earnshaw
2023-01-03 14:13       ` Martin Storsjö
2023-01-03 14:54         ` Tamar Christina
2023-01-03 15:51           ` Martin Storsjö
2023-01-03 15:57             ` Tamar Christina
2023-01-03 18:21           ` Mark Harmstone
2023-01-03 18:33             ` Andrew Pinski
2023-01-03 19:07               ` Mark Harmstone
2023-01-03 19:41               ` Tamar Christina
2023-01-03 20:05                 ` Martin Storsjö
2023-01-04  2:38                   ` Mark Harmstone [this message]
2023-01-04  9:51                     ` Nick Clifton
2023-01-04 10:25                       ` Martin Storsjö
2023-01-04 10:35                         ` Tamar Christina
2023-01-04 11:00                           ` Martin Storsjö
2023-01-04 11:08                             ` Tamar Christina
2023-01-04 11:36                               ` Martin Storsjö
2023-01-04 15:02                                 ` Nick Clifton
2023-01-05  2:33                                   ` Mark Harmstone
2023-01-05 11:01                                     ` Nick Clifton
2023-01-05 10:45                                   ` Tamar Christina
2023-01-03 14:14       ` Tamar Christina
2023-01-03 12:53   ` Martin Storsjö

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=367317ba-108e-fde8-98d2-0be5146f28fa@harmstone.com \
    --to=mark@harmstone.com \
    --cc=Richard.Earnshaw@foss.arm.com \
    --cc=Tamar.Christina@arm.com \
    --cc=binutils@sourceware.org \
    --cc=martin@martin.st \
    --cc=nickc@redhat.com \
    --cc=nightstrike@gmail.com \
    --cc=pinskia@gmail.com \
    --cc=wej22007@outlook.com \
    --cc=zac.walker@linaro.org \
    /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).