[PATCH] D29356: [sanitizer] Support SANITIZER_INTERCEPTOR_HOOKS on Darwin

Justin Bogner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 31 17:09:40 PST 2017


bogner added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_common_interceptors.inc:49
-  do {                                                                         \
-    if (f)                                                                     \
-      f(__VA_ARGS__);                                                          \
----------------
kcc wrote:
> Why did you remove the 'if'? 
SANITIZER_INTERFACE_WEAK_DEF defines an empty default definition, so it would always be true.


https://reviews.llvm.org/D29356





More information about the llvm-commits mailing list