[PATCH] D87095: [Triple][MachO] Define "arm64e", an AArch64 subarch for Pointer Auth.

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 4 01:00:45 PDT 2020


jhenderson added inline comments.


================
Comment at: llvm/lib/Object/MachOObjectFile.cpp:2745
 ArrayRef<StringRef> MachOObjectFile::getValidArchs() {
-  static const std::array<StringRef, 17> validArchs = {{
+  static const std::array<StringRef, 18> validArchs = {{
       "i386",   "x86_64", "x86_64h",  "armv4t",  "arm",    "armv5e",
----------------
Nit: it's probably worth fixing these two Linter issues, since this function is small anyway.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87095



More information about the llvm-commits mailing list