[compiler-rt] r317958 - [ubsan] Fix ubsan tests broken by linking as C instead of C++

Vitaly Buka via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 10 17:01:09 PST 2017


Author: vitalybuka
Date: Fri Nov 10 17:01:09 2017
New Revision: 317958

URL: http://llvm.org/viewvc/llvm-project?rev=317958&view=rev
Log:
[ubsan] Fix ubsan tests broken by linking as C instead of C++

Modified:
    compiler-rt/trunk/test/sanitizer_common/TestCases/Linux/ill.cc
    compiler-rt/trunk/test/sanitizer_common/TestCases/Posix/fpe.cc

Modified: compiler-rt/trunk/test/sanitizer_common/TestCases/Linux/ill.cc
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/sanitizer_common/TestCases/Linux/ill.cc?rev=317958&r1=317957&r2=317958&view=diff
==============================================================================
--- compiler-rt/trunk/test/sanitizer_common/TestCases/Linux/ill.cc (original)
+++ compiler-rt/trunk/test/sanitizer_common/TestCases/Linux/ill.cc Fri Nov 10 17:01:09 2017
@@ -1,7 +1,7 @@
 // Test the handle_sigill option.
 
 // clang-format off
-// RUN: %clang %s -o %t -O1
+// RUN: %clangxx %s -o %t -O1
 // RUN:                                not --crash %run %t 2>&1 | FileCheck --check-prefix=CHECK0 %s
 // RUN: %env_tool_opts=handle_sigill=0 not --crash %run %t 2>&1 | FileCheck --check-prefix=CHECK0 %s
 // RUN: %env_tool_opts=handle_sigill=1 not         %run %t 2>&1 | FileCheck --check-prefix=CHECK1 %s
@@ -9,8 +9,7 @@
 
 // FIXME: implement in other sanitizers.
 // XFAIL: tsan
-// XFAIL: ubsan
-//
+
 // FIXME: seems to fail on ARM
 // REQUIRES: x86_64-target-arch
 #include <assert.h>

Modified: compiler-rt/trunk/test/sanitizer_common/TestCases/Posix/fpe.cc
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/sanitizer_common/TestCases/Posix/fpe.cc?rev=317958&r1=317957&r2=317958&view=diff
==============================================================================
--- compiler-rt/trunk/test/sanitizer_common/TestCases/Posix/fpe.cc (original)
+++ compiler-rt/trunk/test/sanitizer_common/TestCases/Posix/fpe.cc Fri Nov 10 17:01:09 2017
@@ -1,12 +1,12 @@
 // Test the handle_sigfpe option.
-// RUN: %clang %s -o %t
+// RUN: %clangxx %s -o %t
 // RUN:                               not         %run %t 2>&1 | FileCheck --check-prefix=CHECK1 %s
 // RUN: %env_tool_opts=handle_sigfpe=0 not --crash %run %t 2>&1 | FileCheck --check-prefix=CHECK0 %s
 // RUN: %env_tool_opts=handle_sigfpe=1 not         %run %t 2>&1 | FileCheck --check-prefix=CHECK1 %s
+
 // FIXME: implement in other sanitizers, not just asan.
 // XFAIL: tsan
-// XFAIL: ubsan
-//
+
 // FIXME: seems to fail on ARM
 // REQUIRES: x86_64-target-arch
 #include <assert.h>




More information about the llvm-commits mailing list