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