[compiler-rt] 015994b - [ASan][test-only] Remove superfluous guards in stack_container_dynamic_lib.c (#188469)

[via llvm-commits]

Author: Dan Blackwell
Date: 2026-03-27T10:10:09Z
New Revision: 015994b7f9bce814c8851b7dd6d1718fc8dbb514

URL: https://github.com/llvm/llvm-project/commit/015994b7f9bce814c8851b7dd6d1718fc8dbb514
DIFF: https://github.com/llvm/llvm-project/commit/015994b7f9bce814c8851b7dd6d1718fc8dbb514.diff

LOG: [ASan][test-only] Remove superfluous guards in stack_container_dynamic_lib.c (#188469)

As noted in https://github.com/llvm/llvm-project/pull/188406 comments,
the documentation recommends guarding only with
__has_feature(address_sanitizer). This patch updates the test to follow
the same pattern by removing the
__SANITIZER_DISABLE_CONTAINER_OVERFLOW__ checks. Having this macro
defined results in the common_interface_defs.h header defining the
contiguous container functions as no-ops anyway.

This is a followup to https://github.com/llvm/llvm-project/pull/188406.

Added: 
    

Modified: 
    compiler-rt/test/asan/TestCases/stack_container_dynamic_lib.c

Removed: 
    


################################################################################
diff  --git a/compiler-rt/test/asan/TestCases/stack_container_dynamic_lib.c b/compiler-rt/test/asan/TestCases/stack_container_dynamic_lib.c
index ccb6f12228acf..e341f55be0e74 100644
--- a/compiler-rt/test/asan/TestCases/stack_container_dynamic_lib.c
+++ b/compiler-rt/test/asan/TestCases/stack_container_dynamic_lib.c
@@ -33,8 +33,7 @@ struct Stack {
 
 static void init(struct Stack *s) {
   s->size = 0;
-#if __has_feature(address_sanitizer) &&                                        \
-    !defined(__SANITIZER_DISABLE_CONTAINER_OVERFLOW__)
+#if __has_feature(address_sanitizer)
   // Mark entire storage as unaddressable initially
   __sanitizer_annotate_contiguous_container(s->data, s->data + 5, s->data + 5,
                                             s->data);
@@ -42,8 +41,7 @@ static void init(struct Stack *s) {
 }
 
 static void destroy(struct Stack *s) {
-#if __has_feature(address_sanitizer) &&                                        \
-    !defined(__SANITIZER_DISABLE_CONTAINER_OVERFLOW__)
+#if __has_feature(address_sanitizer)
   __sanitizer_annotate_contiguous_container(s->data, s->data + 5,
                                             s->data + s->size, s->data + 5);
 #endif
@@ -51,8 +49,7 @@ static void destroy(struct Stack *s) {
 
 static void push(struct Stack *s, int value) {
   assert(s->size < 5 && "Stack overflow");
-#if __has_feature(address_sanitizer) &&                                        \
-    !defined(__SANITIZER_DISABLE_CONTAINER_OVERFLOW__)
+#if __has_feature(address_sanitizer)
   __sanitizer_annotate_contiguous_container(
       s->data, s->data + 5, s->data + s->size, s->data + s->size + 1);
 #endif
@@ -62,8 +59,7 @@ static void push(struct Stack *s, int value) {
 static int pop(struct Stack *s) {
   assert(s->size > 0 && "Cannot pop from empty stack");
   int result = s->data[--s->size];
-#if __has_feature(address_sanitizer) &&                                        \
-    !defined(__SANITIZER_DISABLE_CONTAINER_OVERFLOW__)
+#if __has_feature(address_sanitizer)
   __sanitizer_annotate_contiguous_container(
       s->data, s->data + 5, s->data + s->size + 1, s->data + s->size);
 #endif