[libcxx-commits] [PATCH] D131484: [libcxx] [test] Split the TEST_HAS_SANITIZERS define into TEST_IS_SLOW

Mark de Wever via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Aug 18 08:50:44 PDT 2022


Mordante added inline comments.


================
Comment at: libcxx/test/support/test_macros.h:190
 #define TEST_HAS_SANITIZERS
+#define TEST_IS_SLOW
 #endif
----------------
I think the name `TEST_IS_SLOW` is a bit generic, we have slow tests that don't use this. (*cough* format)
How about `TEST_IS_EXECUTED_IN_A_SLOW_ENVIRONMENT` or something along these lines?



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131484



More information about the libcxx-commits mailing list