Skip to content

Conversation

@padriff
Copy link
Contributor

@padriff padriff commented Nov 19, 2025

New test stack_container_dynamic_lib.cpp has errors under gcc.

Require clang while better fix is investigated

@llvmbot
Copy link
Member

llvmbot commented Nov 19, 2025

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: Paddy McDonald (padriff)

Changes

New test stack_container_dynamic_lib.cpp has errors under gcc.

Require clang while better fix is investigated


Full diff: https://github.com/llvm/llvm-project/pull/168792.diff

1 Files Affected:

  • (modified) compiler-rt/test/asan/TestCases/stack_container_dynamic_lib.cpp (+3)
diff --git a/compiler-rt/test/asan/TestCases/stack_container_dynamic_lib.cpp b/compiler-rt/test/asan/TestCases/stack_container_dynamic_lib.cpp
index e17c537f4ac1c..1321796759189 100644
--- a/compiler-rt/test/asan/TestCases/stack_container_dynamic_lib.cpp
+++ b/compiler-rt/test/asan/TestCases/stack_container_dynamic_lib.cpp
@@ -2,6 +2,9 @@
 // in order to handle uninstrumented libraries
 // UNSUPPORTED: target={{.*windows-msvc.*}}
 
+// FIXME: Fix this test under GCC.
+// REQUIRES: Clang
+
 // Mimic a closed-source library compiled without ASan
 // RUN: %clangxx_asan -fno-sanitize=address -DSHARED_LIB %s %fPIC -shared -o %t-so.so
 

@padriff
Copy link
Contributor Author

padriff commented Nov 19, 2025

@dyung - disabling the test under gcc for now to allow me to see if I can figure out how to get it working in that environment.

If this looks OK, can you please merge

@github-actions
Copy link

🐧 Linux x64 Test Results

  • 5824 tests passed
  • 1319 tests skipped

@vitalybuka vitalybuka merged commit ff39d59 into llvm:main Nov 20, 2025
12 of 13 checks passed
aadeshps-mcw pushed a commit to aadeshps-mcw/llvm-project that referenced this pull request Nov 26, 2025
New test stack_container_dynamic_lib.cpp has errors under gcc.

Require clang while better fix is investigated
Priyanshu3820 pushed a commit to Priyanshu3820/llvm-project that referenced this pull request Nov 26, 2025
New test stack_container_dynamic_lib.cpp has errors under gcc.

Require clang while better fix is investigated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants