[PATCH] D37781: [test] Enable LeakSanitizer on 64-bit Darwin ASan llvm builds

[Francis Ricci via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL314535: [test] Enable LeakSanitizer on 64-bit Darwin ASan llvm builds (authored by fjricci).

Changed prior to commit:
  https://reviews.llvm.org/D37781?vs=117029&id=117170#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D37781

Files:
  llvm/trunk/test/tools/lto/lit.local.cfg
  llvm/trunk/utils/lit/lit/llvm/config.py


Index: llvm/trunk/utils/lit/lit/llvm/config.py
===================================================================
--- llvm/trunk/utils/lit/lit/llvm/config.py
+++ llvm/trunk/utils/lit/lit/llvm/config.py
@@ -75,6 +75,9 @@
             features.add("long_tests")
 
         if target_triple:
+            if re.match(r'^x86_64.*-apple', target_triple):
+                if 'address' in sanitizers:
+                    self.with_environment('ASAN_OPTIONS', 'detect_leaks=1', append_path=True)
             if re.match(r'^x86_64.*-linux', target_triple):
                 features.add("x86_64-linux")
             if re.match(r'.*-win32$', target_triple):
Index: llvm/trunk/test/tools/lto/lit.local.cfg
===================================================================
--- llvm/trunk/test/tools/lto/lit.local.cfg
+++ llvm/trunk/test/tools/lto/lit.local.cfg
@@ -1,2 +1,6 @@
 if not ('ld64_plugin' in config.available_features and 'X86' in config.root.targets):
-   config.unsupported = True
+    config.unsupported = True
+
+# These tests invoke ld64 from the system, which is not leak-free
+if "Address" in config.llvm_use_sanitizer:
+    config.environment['ASAN_OPTIONS'] = 'detect_leaks=0'


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37781.117170.patch
Type: text/x-patch
Size: 1192 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170929/931ccc76/attachment.bin>