[PATCH] D39798: [Sanitizers] Use SANITIZER_* macros in lib/interception

Rainer Orth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 8 06:06:40 PST 2017


ro created this revision.
Herald added subscribers: fedor.sergeev, krytarowski, srhines.

Unlike the rest of the sanitizer code, lib/interception uses native macros like __linux__
to check for specific targets instead of the common ones like SANITIZER_LINUX.

When working on the Solaris port of the sanitizers, the current style was found to not
only be inconsistent, but clumsy to use because the canonical way to check for Solaris
is to check for __sun__ && __svr4__ which is a mouthful.

Therefore, this patch switches to use SANITIZER_* macros instead.

Tested on x86_64-pc-linux-gnu.


https://reviews.llvm.org/D39798

Files:
  lib/interception/interception.h
  lib/interception/interception_linux.cc
  lib/interception/interception_linux.h
  lib/interception/interception_mac.cc
  lib/interception/interception_mac.h
  lib/interception/interception_type_test.cc
  lib/interception/interception_win.cc
  lib/interception/interception_win.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D39798.122080.patch
Type: text/x-patch
Size: 12676 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171108/81d81055/attachment.bin>


More information about the llvm-commits mailing list