[PATCH] D63601: [Sanitizers] Remove clang_rt.sancov_{begin,end} on Solaris

Rainer Orth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 8 15:10:01 PDT 2019


This revision was automatically updated to reflect the committed changes.
Closed by commit rL365399: [Sanitizers] Remove clang_rt.sancov_{begin,end} on Solaris (authored by ro, committed by ).
Herald added a subscriber: delcypher.

Changed prior to commit:
  https://reviews.llvm.org/D63601?vs=205806&id=208516#toc

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63601/new/

https://reviews.llvm.org/D63601

Files:
  compiler-rt/trunk/lib/sanitizer_common/CMakeLists.txt
  compiler-rt/trunk/lib/sanitizer_common/sancov_begin.S
  compiler-rt/trunk/lib/sanitizer_common/sancov_end.S


Index: compiler-rt/trunk/lib/sanitizer_common/CMakeLists.txt
===================================================================
--- compiler-rt/trunk/lib/sanitizer_common/CMakeLists.txt
+++ compiler-rt/trunk/lib/sanitizer_common/CMakeLists.txt
@@ -273,38 +273,6 @@
   CFLAGS ${SANITIZER_NO_WEAK_HOOKS_CFLAGS}
   DEFS ${SANITIZER_COMMON_DEFINITIONS})
 
-if(OS_NAME MATCHES "SunOS")
-  # Solaris ld doesn't support the non-standard GNU ld extension of adding
-  # __start_SECNAME and __stop_SECNAME labels to sections whose names are
-  # valid C identifiers.  Instead we add our own definitions for the
-  # __sancov_guards section.
-  add_compiler_rt_object_libraries(SancovBegin
-    ARCHS ${SANITIZER_COMMON_SUPPORTED_ARCH}
-    SOURCES sancov_begin.S
-    CFLAGS ${SANITIZER_CFLAGS}
-    DEFS ${SANITIZER_COMMON_DEFINITIONS})
-
-  add_compiler_rt_runtime(clang_rt.sancov_begin
-    STATIC
-    ARCHS ${SANITIZER_COMMON_SUPPORTED_ARCH}
-    OBJECT_LIBS SancovBegin
-    CFLAGS ${SANITIZER_CFLAGS}
-    DEFS ${SANITIZER_COMMON_DEFINITIONS})
-
-  add_compiler_rt_object_libraries(SancovEnd
-    ARCHS ${SANITIZER_COMMON_SUPPORTED_ARCH}
-    SOURCES sancov_end.S
-    CFLAGS ${SANITIZER_CFLAGS}
-    DEFS ${SANITIZER_COMMON_DEFINITIONS})
-
-  add_compiler_rt_runtime(clang_rt.sancov_end
-    STATIC
-    ARCHS ${SANITIZER_COMMON_SUPPORTED_ARCH}
-    OBJECT_LIBS SancovEnd
-    CFLAGS ${SANITIZER_CFLAGS}
-    DEFS ${SANITIZER_COMMON_DEFINITIONS})
-endif()
-
 if(WIN32)
   add_compiler_rt_object_libraries(SanitizerCommonWeakInterception
     ${SANITIZER_COMMON_SUPPORTED_OS}
Index: compiler-rt/trunk/lib/sanitizer_common/sancov_begin.S
===================================================================
--- compiler-rt/trunk/lib/sanitizer_common/sancov_begin.S
+++ compiler-rt/trunk/lib/sanitizer_common/sancov_begin.S
@@ -1,5 +0,0 @@
-	.type		__start___sancov_guards, at object
-	.globl		__start___sancov_guards
-        .section        __sancov_guards,"aw", at progbits
-        .p2align        2
-__start___sancov_guards:
Index: compiler-rt/trunk/lib/sanitizer_common/sancov_end.S
===================================================================
--- compiler-rt/trunk/lib/sanitizer_common/sancov_end.S
+++ compiler-rt/trunk/lib/sanitizer_common/sancov_end.S
@@ -1,5 +0,0 @@
-	.type		__stop___sancov_guards, at object
-	.globl		__stop___sancov_guards
-        .section        __sancov_guards,"aw", at progbits
-        .p2align        2
-__stop___sancov_guards:


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63601.208516.patch
Type: text/x-patch
Size: 2456 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190708/75af7042/attachment.bin>


More information about the llvm-commits mailing list