[llvm-commits] [compiler-rt] r168169 - in /compiler-rt/trunk/lib: CMakeLists.txt ubsan/CMakeLists.txt ubsan/lit_tests/Integer/add-overflow.cpp ubsan/lit_tests/Integer/sub-overflow.cpp ubsan/lit_tests/TypeCheck/vptr.cpp ubsan/lit_tests/lit.cfg

Alexey Samsonov samsonov at google.com
Fri Nov 16 04:57:07 PST 2012


Author: samsonov
Date: Fri Nov 16 06:57:07 2012
New Revision: 168169

URL: http://llvm.org/viewvc/llvm-project?rev=168169&view=rev
Log:
UBSan: enable CMake build on Darwin. Fix two failing tests and disable the test which produces linker errors on Darwin.

Modified:
    compiler-rt/trunk/lib/CMakeLists.txt
    compiler-rt/trunk/lib/ubsan/CMakeLists.txt
    compiler-rt/trunk/lib/ubsan/lit_tests/Integer/add-overflow.cpp
    compiler-rt/trunk/lib/ubsan/lit_tests/Integer/sub-overflow.cpp
    compiler-rt/trunk/lib/ubsan/lit_tests/TypeCheck/vptr.cpp
    compiler-rt/trunk/lib/ubsan/lit_tests/lit.cfg

Modified: compiler-rt/trunk/lib/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/CMakeLists.txt?rev=168169&r1=168168&r2=168169&view=diff
==============================================================================
--- compiler-rt/trunk/lib/CMakeLists.txt (original)
+++ compiler-rt/trunk/lib/CMakeLists.txt Fri Nov 16 06:57:07 2012
@@ -6,12 +6,11 @@
   add_subdirectory(asan)
   add_subdirectory(interception)
   add_subdirectory(sanitizer_common)
+  add_subdirectory(ubsan)
 endif()
 if("${CMAKE_SYSTEM_NAME}" STREQUAL "Linux")
   # ThreadSanitizer is supported on Linux only.
   add_subdirectory(tsan)
-  # UndefinedBehaviorSanitizer has been tested on Linux only.
-  add_subdirectory(ubsan)
 endif()
 
 # FIXME: Add support for the profile library.

Modified: compiler-rt/trunk/lib/ubsan/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/ubsan/CMakeLists.txt?rev=168169&r1=168168&r2=168169&view=diff
==============================================================================
--- compiler-rt/trunk/lib/ubsan/CMakeLists.txt (original)
+++ compiler-rt/trunk/lib/ubsan/CMakeLists.txt Fri Nov 16 06:57:07 2012
@@ -14,27 +14,41 @@
 
 set(UBSAN_RUNTIME_LIBRARIES)
 
-if(CAN_TARGET_X86_64)
-  add_library(clang_rt.ubsan-x86_64 STATIC
+if(APPLE)
+  # Build universal binary on APPLE.
+  add_library(clang_rt.ubsan_osx STATIC
     ${UBSAN_SOURCES}
-    $<TARGET_OBJECTS:RTSanitizerCommon.x86_64>
+    $<TARGET_OBJECTS:RTSanitizerCommon.osx>
     )
-  set_target_compile_flags(clang_rt.ubsan-x86_64
-    ${UBSAN_CFLAGS} ${TARGET_X86_64_CFLAGS}
-    )
-  list(APPEND UBSAN_RUNTIME_LIBRARIES clang_rt.ubsan-x86_64)
+  set_target_compile_flags(clang_rt.ubsan_osx ${UBSAN_CFLAGS})
+  filter_available_targets(UBSAN_TARGETS x86_64 i386)
+  set_target_properties(clang_rt.ubsan_osx PROPERTIES
+    OSX_ARCHITECTURES "${UBSAN_TARGETS}")
+  list(APPEND UBSAN_RUNTIME_LIBRARIES clang_rt.ubsan_osx)
+else()
+  # Build separate libraries for each target.
+  if(CAN_TARGET_X86_64)
+    add_library(clang_rt.ubsan-x86_64 STATIC
+      ${UBSAN_SOURCES}
+      $<TARGET_OBJECTS:RTSanitizerCommon.x86_64>
+      )
+    set_target_compile_flags(clang_rt.ubsan-x86_64
+      ${UBSAN_CFLAGS} ${TARGET_X86_64_CFLAGS}
+      )
+    list(APPEND UBSAN_RUNTIME_LIBRARIES clang_rt.ubsan-x86_64)
+  endif()
+  if(CAN_TARGET_I386)
+    add_library(clang_rt.ubsan-i386 STATIC
+      ${UBSAN_SOURCES}
+      $<TARGET_OBJECTS:RTSanitizerCommon.i386>
+      )
+    set_target_compile_flags(clang_rt.ubsan-i386
+      ${UBSAN_CFLAGS} ${TARGET_I386_CFLAGS}
+      )
+    list(APPEND UBSAN_RUNTIME_LIBRARIES clang_rt.ubsan-i386)
+  endif()
 endif()
 
-if(CAN_TARGET_I386)
-  add_library(clang_rt.ubsan-i386 STATIC
-    ${UBSAN_SOURCES}
-    $<TARGET_OBJECTS:RTSanitizerCommon.i386>
-    )
-  set_target_compile_flags(clang_rt.ubsan-i386
-    ${UBSAN_CFLAGS} ${TARGET_I386_CFLAGS}
-    )
-  list(APPEND UBSAN_RUNTIME_LIBRARIES clang_rt.ubsan-i386)
-endif()
 
 set_property(TARGET ${UBSAN_RUNTIME_LIBRARIES} APPEND PROPERTY
   COMPILE_DEFINITIONS ${UBSAN_COMMON_DEFINITIONS})

Modified: compiler-rt/trunk/lib/ubsan/lit_tests/Integer/add-overflow.cpp
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/ubsan/lit_tests/Integer/add-overflow.cpp?rev=168169&r1=168168&r2=168169&view=diff
==============================================================================
--- compiler-rt/trunk/lib/ubsan/lit_tests/Integer/add-overflow.cpp (original)
+++ compiler-rt/trunk/lib/ubsan/lit_tests/Integer/add-overflow.cpp Fri Nov 16 06:57:07 2012
@@ -17,7 +17,7 @@
 
 #ifdef ADD_I64
   (void)(int64_t(8000000000000000000ll) + int64_t(2000000000000000000ll));
-  // CHECK-ADD_I64: 8000000000000000000 + 2000000000000000000 cannot be represented in type 'long'
+  // CHECK-ADD_I64: 8000000000000000000 + 2000000000000000000 cannot be represented in type '{{long( long)?}}'
 #endif
 
 #ifdef ADD_I128

Modified: compiler-rt/trunk/lib/ubsan/lit_tests/Integer/sub-overflow.cpp
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/ubsan/lit_tests/Integer/sub-overflow.cpp?rev=168169&r1=168168&r2=168169&view=diff
==============================================================================
--- compiler-rt/trunk/lib/ubsan/lit_tests/Integer/sub-overflow.cpp (original)
+++ compiler-rt/trunk/lib/ubsan/lit_tests/Integer/sub-overflow.cpp Fri Nov 16 06:57:07 2012
@@ -16,7 +16,7 @@
 
 #ifdef SUB_I64
   (void)(int64_t(-8000000000000000000ll) - int64_t(2000000000000000000ll));
-  // CHECK-SUB_I64: -8000000000000000000 - 2000000000000000000 cannot be represented in type 'long'
+  // CHECK-SUB_I64: -8000000000000000000 - 2000000000000000000 cannot be represented in type '{{long( long)?}}'
 #endif
 
 #ifdef SUB_I128

Modified: compiler-rt/trunk/lib/ubsan/lit_tests/TypeCheck/vptr.cpp
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/ubsan/lit_tests/TypeCheck/vptr.cpp?rev=168169&r1=168168&r2=168169&view=diff
==============================================================================
--- compiler-rt/trunk/lib/ubsan/lit_tests/TypeCheck/vptr.cpp (original)
+++ compiler-rt/trunk/lib/ubsan/lit_tests/TypeCheck/vptr.cpp Fri Nov 16 06:57:07 2012
@@ -8,6 +8,9 @@
 // RUN: %t mV 2>&1 | FileCheck %s --check-prefix=CHECK-MEMBER
 // RUN: %t fV 2>&1 | FileCheck %s --check-prefix=CHECK-MEMFUN
 
+// FIXME: This test produces linker errors on Darwin.
+// XFAIL: darwin
+
 struct S {
   S() : a(0) {}
   ~S() {}

Modified: compiler-rt/trunk/lib/ubsan/lit_tests/lit.cfg
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/ubsan/lit_tests/lit.cfg?rev=168169&r1=168168&r2=168169&view=diff
==============================================================================
--- compiler-rt/trunk/lib/ubsan/lit_tests/lit.cfg (original)
+++ compiler-rt/trunk/lib/ubsan/lit_tests/lit.cfg Fri Nov 16 06:57:07 2012
@@ -59,6 +59,7 @@
 # Default test suffixes.
 config.suffixes = ['.c', '.cc', '.cpp']
 
-# UndefinedBehaviorSanitizer tests are currently supported on Linux only.
-if config.host_os not in ['Linux']:
+# UndefinedBehaviorSanitizer tests are currently supported on
+# Linux and Darwin only.
+if config.host_os not in ['Linux', 'Darwin']:
   config.unsupported = True





More information about the llvm-commits mailing list