SjoerdMeijer added inline comments.
================ Comment at: clang/test/Driver/aarch64-cpus.c:298 +// RUN: %clang -target aarch64 -mcpu=tsv110 -### -c %s 2>&1 | FileCheck -check-prefix=TSV110 %s +// RUN: %clang -target aarch64 -mlittle-endian -mcpu=tsv110 -### -c %s 2>&1 | FileCheck -check-prefix=TSV110 %s ---------------- This is unrelated to the scheduling model. Looks like good tests to me, so please go ahead and commit this separately, no need for another review. ================ Comment at: llvm/lib/Target/AArch64/AArch64SchedTSV110Details.td:1 +//==- AArch64SchedTSV110Details.td - TSV110 Scheduling Defs -*- tablegen -*-==// +// ---------------- Just a nit/question on this new file: is there a benefit of having this separately? If there's none, just merge into AArch64SchedTSV110.td, as that would be more consistent with other schedmodels? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89972/new/ https://reviews.llvm.org/D89972 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
