[libcxx-commits] [PATCH] D144632: [libcxx] Split modules_include test to reduce overall runtime

[David Spickett via Phabricator via libcxx-commits]

DavidSpickett added reviewers: ldionne, Mordante, philnik.
DavidSpickett added inline comments.


================
Comment at: libcxx/test/libcxx/modules_include_1.sh.cpp:39
+
+per_test = len(public_headers) // 4
+
----------------
// otherwise you get a float, and you can't slice with a float.


================
Comment at: libcxx/test/libcxx/modules_include_1.sh.cpp:59
+// RUN: %{cxx} %s %{flags} %{compile_flags} -fmodules -fcxx-modules -fmodules-cache-path=%t -fsyntax-only -DTEST_0
+#if defined(TEST_0)
+#include <algorithm>
----------------
If we want these numbers to still match the index of the header in public_headers, that can be arranged.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144632