[PATCH] D121093: [Driver][AArch64] Split up aarch64-cpus.c test further

Simon Tatham via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 7 02:46:16 PST 2022


simon_tatham added inline comments.


================
Comment at: clang/test/Driver/aarch64-cpus.c:2
+// Check target CPUs are correctly passed.
+// TODO: The files should be split up by categories, e.g. by architecture versions, to avoid excessive test
 // times for large single test files.
----------------
Tiniest nit ever: what files is this comment talking about?

(In the previous version, it followed on from a previous comment that gave the name of the file containing other half of the tests in question. Now it doesn't, so the wording is no longer clear.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121093



More information about the cfe-commits mailing list