[PATCH] D42156: Force lit to execute the ASan and TSan tests on iOS devices sequentially

Kuba (Brecka) Mracek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 17 20:23:42 PST 2018


kubamracek added inline comments.


================
Comment at: test/asan/lit.cfg:218
+    config.parallelism_group = "darwin-64bit-sanitizer"
+  elif config.ios:
+    config.parallelism_group = "darwin-ios-device-sanitizer"
----------------
Should this be `config.ios and not config.iossim`? Note that `config.ios` is "true" even for the simulator.


================
Comment at: test/lit.common.cfg:297
+# tests on iOS devices.
+if config.host_os == 'Darwin' and config.ios:
+  lit_config.warning("iOS device test cases being run sequentially")
----------------
drop the "and config.ios" part of the condition


https://reviews.llvm.org/D42156





More information about the llvm-commits mailing list