diff options
author | Nico Weber <nicolasweber@gmx.de> | 2016-01-22 20:26:10 +0000 |
---|---|---|
committer | Nico Weber <nicolasweber@gmx.de> | 2016-01-22 20:26:10 +0000 |
commit | f1e6d2601dcb3f844cdb1e147e405a94c0c613fe (patch) | |
tree | 4f7195ece0efac373f2431d4bd7188a272699665 /test/ubsan | |
parent | 76720429ce45a4b87d7aef94555fddad1555d56c (diff) |
When building without DIA SDK, don't set suppressions.cpp fail
This test requires llvm-symbolizer to be able to convert a stack
address into a function name. It is only able to do this if the
DIA SDK was found at cmake time. Add a lit feature for this,
and let the test depend on it.
See also discussion in D15363.
git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@258545 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/ubsan')
-rw-r--r-- | test/ubsan/TestCases/Integer/suppressions.cpp | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/test/ubsan/TestCases/Integer/suppressions.cpp b/test/ubsan/TestCases/Integer/suppressions.cpp index 5203efd01..a9e660111 100644 --- a/test/ubsan/TestCases/Integer/suppressions.cpp +++ b/test/ubsan/TestCases/Integer/suppressions.cpp @@ -1,11 +1,9 @@ -// XFAIL: win32 -// This test fails on Windows if the environment was set up by SetEnv.cmd from -// the Windows SDK. If it's set up via vcvarsall.bat, it passes. -// FIXME: Figure out how to make this reliably pass on Windows. -// test/asan/TestCases/suppressions-interceptor.cc will need the same fix. - // RUN: %clangxx -fsanitize=integer -g0 %s -o %t +// Suppression by symbol name (unsigned-integer-overflow:do_overflow below) +// requires the compiler-rt runtime to be able to symbolize stack addresses. +// REQUIRES: can-symbolize + // Fails without any suppression. // RUN: %env_ubsan_opts=halt_on_error=1 not %run %t 2>&1 | FileCheck %s @@ -36,4 +34,3 @@ int main() { do_overflow(); return 0; } - |