Commit 544a8c33 authored by Igor Sugak's avatar Igor Sugak Committed by Facebook Github Bot

remove UNDEFINED_SANITIZER use

Summary: Assume, when UBSAN is used, it is always used on top of ASAN.

Reviewed By: luciang

Differential Revision: D8360231

fbshipit-source-id: 3106cfa53567b18f199e9303ea4c14a464ff03cc
parent dc6ade21
...@@ -105,8 +105,7 @@ ...@@ -105,8 +105,7 @@
* Define a convenience macro to test when ASAN, UBSAN or TSAN sanitizer are * Define a convenience macro to test when ASAN, UBSAN or TSAN sanitizer are
* being used * being used
*/ */
#if defined(FOLLY_SANITIZE_ADDRESS) || defined(FOLLY_SANITIZE_THREAD) || \ #if defined(FOLLY_SANITIZE_ADDRESS) || defined(FOLLY_SANITIZE_THREAD)
defined(UNDEFINED_SANITIZER)
#define FOLLY_SANITIZE 1 #define FOLLY_SANITIZE 1
#endif #endif
......
...@@ -37,8 +37,7 @@ namespace fibers { ...@@ -37,8 +37,7 @@ namespace fibers {
namespace { namespace {
inline FiberManager::Options preprocessOptions(FiberManager::Options opts) { inline FiberManager::Options preprocessOptions(FiberManager::Options opts) {
#if defined(FOLLY_SANITIZE_ADDRESS) || defined(UNDEFINED_SANITIZER) || \ #if defined(FOLLY_SANITIZE_ADDRESS) || defined(FOLLY_SANITIZE_THREAD)
defined(FOLLY_SANITIZE_THREAD)
/* Sanitizers need a lot of extra stack space. /* Sanitizers need a lot of extra stack space.
16x is a conservative estimate, 8x also worked with tests 16x is a conservative estimate, 8x also worked with tests
where it mattered. Note that overallocating here does not necessarily where it mattered. Note that overallocating here does not necessarily
......
...@@ -584,8 +584,8 @@ TEST(ThreadLocal, Fork2) { ...@@ -584,8 +584,8 @@ TEST(ThreadLocal, Fork2) {
// //
// We also cannot run this test unless folly was compiled with PIC support, // We also cannot run this test unless folly was compiled with PIC support,
// since we cannot build thread_local_test_lib.so without PIC. // since we cannot build thread_local_test_lib.so without PIC.
#if defined FOLLY_SANITIZE_ADDRESS || defined UNDEFINED_SANITIZER || \ #if defined FOLLY_SANITIZE_ADDRESS || defined FOLLY_SANITIZE_THREAD || \
defined FOLLY_SANITIZE_THREAD || !defined FOLLY_SUPPORT_SHARED_LIBRARY !defined FOLLY_SUPPORT_SHARED_LIBRARY
#define SHARED_LIBRARY_TEST_NAME DISABLED_SharedLibrary #define SHARED_LIBRARY_TEST_NAME DISABLED_SharedLibrary
#else #else
#define SHARED_LIBRARY_TEST_NAME SharedLibrary #define SHARED_LIBRARY_TEST_NAME SharedLibrary
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment