Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/sys/sys Handle LSan/LLVM and LSan/GCC in __NO_LEAKS



details:   https://anonhg.NetBSD.org/src/rev/dbad9060e3d5
branches:  trunk
changeset: 454610:dbad9060e3d5
user:      kamil <kamil%NetBSD.org@localhost>
date:      Sun Sep 22 23:23:12 2019 +0000

description:
Handle LSan/LLVM and LSan/GCC in __NO_LEAKS

__has_feature(leak_sanitizer) was merged with Clang/LLVM today:
https://reviews.llvm.org/D67719

GCC specific ifdef __SANITIZE_LEAK__ is pending in upstream review...
https://gcc.gnu.org/ml/gcc-patches/2019-09/msg01270.html and can be
rejected as GCC upstream does not see value in feature parity with LLVM
sanitizers. For the time being this will be a NetBSD specific extension.

diffstat:

 sys/sys/cdefs.h |  5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diffs (19 lines):

diff -r bec1530a2580 -r dbad9060e3d5 sys/sys/cdefs.h
--- a/sys/sys/cdefs.h   Sun Sep 22 23:18:53 2019 +0000
+++ b/sys/sys/cdefs.h   Sun Sep 22 23:23:12 2019 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: cdefs.h,v 1.145 2019/09/22 23:13:10 kamil Exp $        */
+/*     $NetBSD: cdefs.h,v 1.146 2019/09/22 23:23:12 kamil Exp $        */
 
 /* * Copyright (c) 1991, 1993
  *     The Regents of the University of California.  All rights reserved.
@@ -345,7 +345,8 @@
 #endif
 
 #if defined(__COVERITY__) ||                                           \
-    __has_feature(address_sanitizer) || defined(__SANITIZE_ADDRESS__)
+    __has_feature(address_sanitizer) || defined(__SANITIZE_ADDRESS__) ||\
+    __has_feature(leak_sanitizer) || defined(__SANITIZE_LEAK__)
 #define        __NO_LEAKS
 #endif
 



Home | Main Index | Thread Index | Old Index