[PATCH] D84770: [llvm-libtool-darwin] Add support for -arch_only

[Shoaib Meenai via Phabricator via llvm-commits]

smeenai added subscribers: cishida, ributzka.
smeenai added a comment.

In D84770#2204147 <https://reviews.llvm.org/D84770#2204147>, @sameerarora101 wrote:

> I think I need to add V8 support under `llvm/include/llvm/TextAPI/MachO/Architecture.def` here:
>
>   ///
>   /// ARM64 architectures sorted by cpu sub type id.
>   ///
>   ARCHINFO(arm64, MachO::CPU_TYPE_ARM64, MachO::CPU_SUBTYPE_ARM64_ALL, 64)
>   ARCHINFO(arm64e, MachO::CPU_TYPE_ARM64, MachO::CPU_SUBTYPE_ARM64E, 64)
>
> What should be the first element for that tuple in case of V8 (`arm64_v8` or `arm64v8` or maybe something else) ? Thanks.

Hmm, what necessitates the TAPI patch? `arm64_v8` seems fine to me, though @ributzka and @cishida should weigh in there.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84770