[PATCH] D151981: [AArch64] Complete the list of extensions supported by .arch and .arch_extension

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 2 03:56:07 PDT 2023


mstorsjo added a comment.

In D151981#4390386 <https://reviews.llvm.org/D151981#4390386>, @tmatheson wrote:

> I would suggest using "rdm" which is:
>
> - consistent with both clang and AArch64TargetParser
> - consistent with the actual feature name in the Arm ARM, which is FEAT_RDM. "ARMv8.1-RDMA" is the legacy feature name.
> - accepted by binutils

Ok, those are good arguments. Although, as binutils still documents it as `rdma`, do you think it'd make sense to support both namings for it? I'd hate to not support the naming that they advertise. I guess it'd be as simple as having two entries in this table.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151981/new/

https://reviews.llvm.org/D151981



More information about the llvm-commits mailing list