[PATCH] D83206: [PATCH] [ARM] Add Cortex-A78 and Cortex-X1 Support for Clang and LLVM

[Mikhail Maltsev via Phabricator via cfe-commits]

miyuki added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64Subtarget.h:72
+    ThunderX3T110,
+    CortexX1,
+    CortexA78
----------------
I think the new CPUs should be grouped with other Cortex-A CPUs. 


================
Comment at: llvm/lib/Target/ARM/ARMSubtarget.h:78
+    Swift,
+    CortexX1,
+    CortexA78
----------------
Please keep the list sorted in alphabetical order.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83206