[PATCH] D53239: [python] [tests] Disable python binding tests when building with LLVM_USE_SANITIZER=Address

2018-10-15 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL344537: [python] [tests] Disable python binding tests under LLVM_USE_SANITIZER=Address (authored by dergachev, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

[PATCH] D53239: [python] [tests] Disable python binding tests when building with LLVM_USE_SANITIZER=Address

2018-10-13 Thread Michał Górny via Phabricator via cfe-commits
mgorny accepted this revision. mgorny added a comment. WFM. Thanks for analyzing the problem. Comment at: bindings/python/tests/CMakeLists.txt:27 +# with ASan. +if((NOT WIN32) AND (NOT LLVM_USE_ASAN)) set_property(GLOBAL APPEND PROPERTY Hmm, I don't think

[PATCH] D53239: [python] [tests] Disable python binding tests when building with LLVM_USE_SANITIZER=Address

2018-10-12 Thread Dan Liew via Phabricator via cfe-commits
delcypher accepted this revision. delcypher added a comment. This revision is now accepted and ready to land. LGTM. But then I would say that because I wrote the patch ... Repository: rC Clang https://reviews.llvm.org/D53239 ___ cfe-commits

[PATCH] D53239: [python] [tests] Disable python binding tests when building with LLVM_USE_SANITIZER=Address

2018-10-12 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ created this revision. NoQ added reviewers: mgorny, delcypher. Herald added a subscriber: cfe-commits. @mgorny: These fail for me on macOS too, if LLVM is configured as cmake ../llvm/ -DCMAKE_BUILD_TYPE=Release -DLLVM_USE_SANITIZER=Address I guess we might have to outright disable them