Skip to content

Commit 8421fa5

Browse files
committed
[ASan] Removed ASAN_SHADOW_SCALE.
This is additional cleanup as follow-up of D104279. Reviewed By: vitalybuka Differential Revision: https://reviews.llvm.org/D115271
1 parent 317dc31 commit 8421fa5

File tree

4 files changed

+3
-32
lines changed

4 files changed

+3
-32
lines changed

compiler-rt/CMakeLists.txt

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -52,25 +52,6 @@ mark_as_advanced(COMPILER_RT_BUILD_ORC)
5252
option(COMPILER_RT_BUILD_GWP_ASAN "Build GWP-ASan, and link it into SCUDO" ON)
5353
mark_as_advanced(COMPILER_RT_BUILD_GWP_ASAN)
5454

55-
set(COMPILER_RT_ASAN_SHADOW_SCALE ""
56-
CACHE STRING "Override the shadow scale to be used in ASan runtime")
57-
58-
if (NOT COMPILER_RT_ASAN_SHADOW_SCALE STREQUAL "")
59-
# Check that the shadow scale value is valid.
60-
if (NOT (COMPILER_RT_ASAN_SHADOW_SCALE GREATER -1 AND
61-
COMPILER_RT_ASAN_SHADOW_SCALE LESS 8))
62-
message(FATAL_ERROR "
63-
Invalid ASan Shadow Scale '${COMPILER_RT_ASAN_SHADOW_SCALE}'.")
64-
endif()
65-
66-
set(COMPILER_RT_ASAN_SHADOW_SCALE_LLVM_FLAG
67-
-mllvm -asan-mapping-scale=${COMPILER_RT_ASAN_SHADOW_SCALE})
68-
set(COMPILER_RT_ASAN_SHADOW_SCALE_DEFINITION
69-
ASAN_SHADOW_SCALE=${COMPILER_RT_ASAN_SHADOW_SCALE})
70-
set(COMPILER_RT_ASAN_SHADOW_SCALE_FLAG
71-
-D${COMPILER_RT_ASAN_SHADOW_SCALE_DEFINITION})
72-
endif()
73-
7455
if(FUCHSIA)
7556
set(COMPILER_RT_HWASAN_WITH_INTERCEPTORS_DEFAULT OFF)
7657
else()

compiler-rt/lib/asan/asan_mapping.h

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -151,14 +151,9 @@
151151
// || `[0x30000000, 0x35ffffff]` || LowShadow ||
152152
// || `[0x00000000, 0x2fffffff]` || LowMem ||
153153

154-
#if defined(ASAN_SHADOW_SCALE)
155-
static const u64 kDefaultShadowScale = ASAN_SHADOW_SCALE;
156-
#else
157-
static const u64 kDefaultShadowScale = 3;
158-
#endif
159-
static const u64 kDefaultShadowSentinel = ~(uptr)0;
154+
#define SHADOW_SCALE 3
160155

161-
#define SHADOW_SCALE kDefaultShadowScale
156+
static const u64 kDefaultShadowSentinel = ~(uptr)0;
162157

163158
#if SANITIZER_FUCHSIA
164159
# define SHADOW_OFFSET_CONST (0)
@@ -204,8 +199,7 @@ static const u64 kDefaultShadowSentinel = ~(uptr)0;
204199
# elif SANITIZER_WINDOWS64
205200
# define SHADOW_OFFSET_DYNAMIC
206201
# else
207-
# define SHADOW_OFFSET_CONST \
208-
0x000000007FFFFFFF & (~0xFFFULL << kDefaultShadowScale)
202+
# define SHADOW_OFFSET_CONST 0x000000007FFFFFFF & (~0xFFFULL << SHADOW_SCALE)
209203
# endif
210204
#endif
211205

compiler-rt/lib/asan/tests/CMakeLists.txt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ set(ASAN_UNITTEST_HEADERS
2323
set(ASAN_UNITTEST_COMMON_CFLAGS
2424
${COMPILER_RT_UNITTEST_CFLAGS}
2525
${COMPILER_RT_GTEST_CFLAGS}
26-
${COMPILER_RT_ASAN_SHADOW_SCALE_LLVM_FLAG}
2726
-I${COMPILER_RT_SOURCE_DIR}/include
2827
-I${COMPILER_RT_SOURCE_DIR}/lib
2928
-I${COMPILER_RT_SOURCE_DIR}/lib/asan
@@ -52,7 +51,6 @@ list(APPEND ASAN_UNITTEST_COMMON_LINK_FLAGS -g)
5251

5352
# Use -D instead of definitions to please custom compile command.
5453
list(APPEND ASAN_UNITTEST_COMMON_CFLAGS
55-
${COMPILER_RT_ASAN_SHADOW_SCALE_FLAG}
5654
-DASAN_HAS_IGNORELIST=1
5755
-DASAN_HAS_EXCEPTIONS=1
5856
-DASAN_UAR=0

compiler-rt/lib/asan/tests/asan_interface_test.cpp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,6 @@ TEST(AddressSanitizerInterface, DeathCallbackTest) {
160160
#define BAD_ACCESS(ptr, offset) \
161161
EXPECT_TRUE(__asan_address_is_poisoned(ptr + offset))
162162

163-
#if !defined(ASAN_SHADOW_SCALE) || ASAN_SHADOW_SCALE == 3
164163
static const char* kUseAfterPoisonErrorMessage = "use-after-poison";
165164

166165
TEST(AddressSanitizerInterface, SimplePoisonMemoryRegionTest) {
@@ -200,7 +199,6 @@ TEST(AddressSanitizerInterface, OverlappingPoisonMemoryRegionTest) {
200199
BAD_ACCESS(array, 96);
201200
free(array);
202201
}
203-
#endif // !defined(ASAN_SHADOW_SCALE) || ASAN_SHADOW_SCALE == 3
204202

205203
TEST(AddressSanitizerInterface, PushAndPopWithPoisoningTest) {
206204
// Vector of capacity 20

0 commit comments

Comments
 (0)