[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