[PATCH] D44432: FreeBSD sanitizer common, intercept couple of more functions

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 16 15:58:08 PDT 2018


vitalybuka added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_platform_interceptors.h:452
+#define SANITIZER_INTERCEPT_STRLCPY \
+  (SI_NETBSD || SI_FREEBSD || SI_OPENBSD || SI_MAC || SI_ANDROID)
 
----------------
devnexen wrote:
> vitalybuka wrote:
> > description says it's FreeBSD
> True but I know those systems support it as well do you prefer FreeBSD only changes or updating the description.
Could you move other OSes in separate patch, please also move the test into posix


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D44432





More information about the llvm-commits mailing list