Skip to content

Commit

Permalink
[Sanitizers][Darwin] Rename Apple macro SANITIZER_MAC -> SANITIZER_APPLE
Browse files Browse the repository at this point in the history
Initial introduction of the new macro before obsoleting the old one - the old name was really confusing.
Also moved SANITIZER_WATCHOS and SANITIZER_TVOS definitions under common #if defined(__APPLE__) block

Differential Revision: https://reviews.llvm.org/D125816
  • Loading branch information
Mariusz Borsa committed May 18, 2022
1 parent dd7233b commit 77014b9
Showing 1 changed file with 22 additions and 19 deletions.
41 changes: 22 additions & 19 deletions compiler-rt/lib/sanitizer_common/sanitizer_platform.h
Expand Up @@ -56,7 +56,9 @@
#endif

#if defined(__APPLE__)
# define SANITIZER_MAC 1
# define SANITIZER_APPLE 1
// SANITIZER_MAC will be deprecated/removed in the future
# define SANITIZER_MAC SANITIZER_APPLE
# include <TargetConditionals.h>
# if TARGET_OS_OSX
# define SANITIZER_OSX 1
Expand All @@ -68,28 +70,29 @@
# else
# define SANITIZER_IOS 0
# endif
# if TARGET_OS_WATCH
# define SANITIZER_WATCHOS 1
# else
# define SANITIZER_WATCHOS 0
# endif
# if TARGET_OS_TV
# define SANITIZER_TVOS 1
# else
# define SANITIZER_TVOS 0
# endif
# if TARGET_OS_SIMULATOR
# define SANITIZER_IOSSIM 1
# else
# define SANITIZER_IOSSIM 0
# endif
#else
# define SANITIZER_MAC 0
# define SANITIZER_APPLE 0
# define SANITIZER_MAC SANITIZER_APPLE
# define SANITIZER_IOS 0
# define SANITIZER_IOSSIM 0
# define SANITIZER_OSX 0
#endif

#if defined(__APPLE__) && TARGET_OS_IPHONE && TARGET_OS_WATCH
# define SANITIZER_WATCHOS 1
#else
# define SANITIZER_WATCHOS 0
#endif

#if defined(__APPLE__) && TARGET_OS_IPHONE && TARGET_OS_TV
# define SANITIZER_TVOS 1
#else
# define SANITIZER_TVOS 0
# define SANITIZER_IOSSIM 0
# define SANITIZER_OSX 0
#endif

#if defined(_WIN32)
Expand Down Expand Up @@ -124,7 +127,7 @@
#endif

#define SANITIZER_POSIX \
(SANITIZER_FREEBSD || SANITIZER_LINUX || SANITIZER_MAC || \
(SANITIZER_FREEBSD || SANITIZER_LINUX || SANITIZER_APPLE || \
SANITIZER_NETBSD || SANITIZER_SOLARIS)

#if __LP64__ || defined(_WIN64)
Expand Down Expand Up @@ -286,7 +289,7 @@
#elif SANITIZER_RISCV64
# define SANITIZER_MMAP_RANGE_SIZE FIRST_32_SECOND_64(1ULL << 32, 1ULL << 38)
#elif defined(__aarch64__)
# if SANITIZER_MAC
# if SANITIZER_APPLE
# if SANITIZER_OSX || SANITIZER_IOSSIM
# define SANITIZER_MMAP_RANGE_SIZE \
FIRST_32_SECOND_64(1ULL << 32, 1ULL << 47)
Expand Down Expand Up @@ -343,7 +346,7 @@
# define MSC_PREREQ(version) 0
#endif

#if SANITIZER_MAC && defined(__x86_64__)
#if SANITIZER_APPLE && defined(__x86_64__)
# define SANITIZER_NON_UNIQUE_TYPEINFO 0
#else
# define SANITIZER_NON_UNIQUE_TYPEINFO 1
Expand Down Expand Up @@ -371,7 +374,7 @@
# define SANITIZER_SUPPRESS_LEAK_ON_PTHREAD_EXIT 0
#endif

#if SANITIZER_FREEBSD || SANITIZER_MAC || SANITIZER_NETBSD || SANITIZER_SOLARIS
#if SANITIZER_FREEBSD || SANITIZER_APPLE || SANITIZER_NETBSD || SANITIZER_SOLARIS
# define SANITIZER_MADVISE_DONTNEED MADV_FREE
#else
# define SANITIZER_MADVISE_DONTNEED MADV_DONTNEED
Expand All @@ -395,7 +398,7 @@
// Enable ability to support sanitizer initialization that is
// compatible with the sanitizer library being loaded via
// `dlopen()`.
#if SANITIZER_MAC
#if SANITIZER_APPLE
# define SANITIZER_SUPPORTS_INIT_FOR_DLOPEN 1
#else
# define SANITIZER_SUPPORTS_INIT_FOR_DLOPEN 0
Expand Down

0 comments on commit 77014b9

Please sign in to comment.