[PATCH] D34644: Only test sanitizers that are built when COMPILER_RT_SANITIZERS_TO_BUILD is used

Francis Ricci via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 27 08:21:11 PDT 2017


fjricci added inline comments.


================
Comment at: test/CMakeLists.txt:60-73
+    compiler_rt_test_runtime(interception)
+
+    compiler_rt_test_runtime(lsan)
+    compiler_rt_test_runtime(ubsan)
+    compiler_rt_test_runtime(sanitizer_common)
+
+    compiler_rt_test_sanitizer(asan)
----------------
compnerd wrote:
> Why not just walk `COMPILER_RT_SANITIZERS_TO_BUILD` rather than doing an intersection for each one?
Will address in a follow-up commit for the build case as well as the test case.


https://reviews.llvm.org/D34644





More information about the llvm-commits mailing list