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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to