[Lldb-commits] [PATCH] D26513: [Test-Suite] Fix all the sanitizer tests to be based on compiler capabilities

Chris Bieneman via lldb-commits lldb-commits at lists.llvm.org
Thu Nov 10 10:22:46 PST 2016


beanz added inline comments.


================
Comment at: packages/Python/lldbsuite/test/functionalities/tsan/basic/TestTsanBasic.py:23
     @skipIfRemote
-    @skipUnlessCompilerRt
     @skipUnlessThreadSanitizer
     def test(self):
----------------
tfiala wrote:
> Seems fine as long as @skipUnlessThreadSanitizer is doing the right type of check like you added for @skipUnlessAddressSanitizer.
The implementation of `skipUnlessAddressSanitizer` is basically copy-paste-simplify-modify of `skipUnlessThreadSanitizer`. They should both have good coverage.


https://reviews.llvm.org/D26513





More information about the lldb-commits mailing list