[llvm] r270510 - test: Be consistent with clang's sanitizer lit config

Justin Bogner via llvm-commits llvm-commits at lists.llvm.org
Mon May 23 16:02:11 PDT 2016


Author: bogner
Date: Mon May 23 18:02:11 2016
New Revision: 270510

URL: http://llvm.org/viewvc/llvm-project?rev=270510&view=rev
Log:
test: Be consistent with clang's sanitizer lit config

The logic that sets up lit features for sanitizers is largely copied
between here and clang, except clang's was fixed some time ago to
handle multiple sanitizers (ie, Asan + Ubsan). This just makes the
code in LLVM consistent with how it's done in clang to avoid any
gotchas by users of this.

Modified:
    llvm/trunk/test/lit.cfg

Modified: llvm/trunk/test/lit.cfg
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/lit.cfg?rev=270510&r1=270509&r2=270510&view=diff
==============================================================================
--- llvm/trunk/test/lit.cfg (original)
+++ llvm/trunk/test/lit.cfg Mon May 23 18:02:11 2016
@@ -339,14 +339,15 @@ if loadable_module:
     config.available_features.add('loadable_module')
 
 # Sanitizers.
-if config.llvm_use_sanitizer == "Address":
+if 'Address' in config.llvm_use_sanitizer:
     config.available_features.add("asan")
-if (config.llvm_use_sanitizer == "Memory" or
-        config.llvm_use_sanitizer == "MemoryWithOrigins"):
+else:
+    config.available_features.add("not_asan")
+if 'Memory' in config.llvm_use_sanitizer:
     config.available_features.add("msan")
 else:
     config.available_features.add("not_msan")
-if config.llvm_use_sanitizer == "Undefined":
+if 'Undefined' in config.llvm_use_sanitizer:
     config.available_features.add("ubsan")
 else:
     config.available_features.add("not_ubsan")




More information about the llvm-commits mailing list