Skip to content

Commit

Permalink
Revert "[NFC][clang][test][asan] Make `instantiation-depth-default.cp…
Browse files Browse the repository at this point in the history
…p` a valid test case under `asan` and `ubsan` configs (#75254)"

Disables the recursive template expansion test under ASan again. This
patch re-enabled this test with sanitizers, but it's started spuriously
failing with a stack overflow again on AArch64+ASan:
https://lab.llvm.org/buildbot/#/builders/239/builds/6363

This reverts commit c458f92.
  • Loading branch information
hctim committed Apr 2, 2024
1 parent b31414b commit bcac3ed
Showing 1 changed file with 11 additions and 5 deletions.
16 changes: 11 additions & 5 deletions clang/test/SemaTemplate/instantiation-depth-default.cpp
Original file line number Diff line number Diff line change
@@ -1,12 +1,18 @@
// RUN: %clang_cc1 -fsyntax-only -verify -ftemplate-backtrace-limit=2 %if {{asan|ubsan}} %{ -Wno-stack-exhausted %} %s
// RUN: %clang_cc1 -fsyntax-only -verify -ftemplate-backtrace-limit=2 %s
//
// FIXME: Disable this test when Clang was built with ASan, because ASan
// increases our per-frame stack usage enough that this test no longer fits
// within our normal stack space allocation.
// UNSUPPORTED: asan
//
// The default stack size on NetBSD is too small for this test.
// UNSUPPORTED: system-netbsd

template<int N, typename T> struct X : X<N+1, T*> {};
// expected-error-re@5 {{recursive template instantiation exceeded maximum depth of 1024{{$}}}}
// expected-note@5 {{instantiation of template class}}
// expected-note@5 {{skipping 1023 contexts in backtrace}}
// expected-note@5 {{use -ftemplate-depth=N to increase recursive template instantiation depth}}
// expected-error-re@11 {{recursive template instantiation exceeded maximum depth of 1024{{$}}}}
// expected-note@11 {{instantiation of template class}}
// expected-note@11 {{skipping 1023 contexts in backtrace}}
// expected-note@11 {{use -ftemplate-depth=N to increase recursive template instantiation depth}}

X<0, int> x; // expected-note {{in instantiation of}}

Expand Down

0 comments on commit bcac3ed

Please sign in to comment.