[PATCH] D37635: Prevent DCE on __lsan_is_turned_off and re-enable test case

Francis Ricci via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 8 12:39:34 PDT 2017


fjricci updated this revision to Diff 114416.
fjricci added a comment.

Add documentation


https://reviews.llvm.org/D37635

Files:
  include/sanitizer/lsan_interface.h
  test/lsan/TestCases/link_turned_off.cc


Index: test/lsan/TestCases/link_turned_off.cc
===================================================================
--- test/lsan/TestCases/link_turned_off.cc
+++ test/lsan/TestCases/link_turned_off.cc
@@ -3,15 +3,13 @@
 // RUN: %clangxx_lsan %s -o %t
 // RUN: %env_lsan_opts=$LSAN_BASE %run %t
 // RUN: %env_lsan_opts=$LSAN_BASE not %run %t foo 2>&1 | FileCheck %s
-//
-// UNSUPPORTED: darwin
 
 #include <sanitizer/lsan_interface.h>
 
 int argc_copy;
 
 extern "C" {
-int __lsan_is_turned_off() {
+int __attribute__((used)) __lsan_is_turned_off() {
   return (argc_copy == 1);
 }
 }
Index: include/sanitizer/lsan_interface.h
===================================================================
--- include/sanitizer/lsan_interface.h
+++ include/sanitizer/lsan_interface.h
@@ -64,6 +64,8 @@
   // for the program it is linked into (if the return value is non-zero). This
   // function must be defined as returning a constant value; any behavior beyond
   // that is unsupported.
+  // To avoid dead stripping, you may need to define this function with
+  // __attribute__((used))
   int __lsan_is_turned_off();
 
   // This function may be optionally provided by the user and should return


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37635.114416.patch
Type: text/x-patch
Size: 1189 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170908/fccb19a9/attachment.bin>


More information about the llvm-commits mailing list