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

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 19 16:47:05 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)
 
----------------
vitalybuka wrote:
> 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
Could you please rebase?


https://reviews.llvm.org/D44432





More information about the llvm-commits mailing list