This revision was automatically updated to reflect the committed changes.
Closed by commit rGc6d799156a0a: [clangd] Skip ClangdVFSTest.TestStackOverflow
when address sanitizer is used (authored by bjope).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D76533/new/
https://reviews.llvm.org/D76533
Files:
clang-tools-extra/clangd/unittests/ClangdTests.cpp
Index: clang-tools-extra/clangd/unittests/ClangdTests.cpp
===================================================================
--- clang-tools-extra/clangd/unittests/ClangdTests.cpp
+++ clang-tools-extra/clangd/unittests/ClangdTests.cpp
@@ -1083,6 +1083,9 @@
Field(&CodeCompletion::Scope, "ns::"))));
}
+// Tests fails when built with asan due to stack overflow. So skip running the
+// test as a workaround.
+#if !defined(__has_feature) || !__has_feature(address_sanitizer)
TEST_F(ClangdVFSTest, TestStackOverflow) {
MockFSProvider FS;
ErrorCheckingCallbacks DiagConsumer;
@@ -1103,6 +1106,7 @@
// overflow
EXPECT_TRUE(DiagConsumer.hadErrorInLastDiags());
}
+#endif
} // namespace
} // namespace clangd
Index: clang-tools-extra/clangd/unittests/ClangdTests.cpp
===================================================================
--- clang-tools-extra/clangd/unittests/ClangdTests.cpp
+++ clang-tools-extra/clangd/unittests/ClangdTests.cpp
@@ -1083,6 +1083,9 @@
Field(&CodeCompletion::Scope, "ns::"))));
}
+// Tests fails when built with asan due to stack overflow. So skip running the
+// test as a workaround.
+#if !defined(__has_feature) || !__has_feature(address_sanitizer)
TEST_F(ClangdVFSTest, TestStackOverflow) {
MockFSProvider FS;
ErrorCheckingCallbacks DiagConsumer;
@@ -1103,6 +1106,7 @@
// overflow
EXPECT_TRUE(DiagConsumer.hadErrorInLastDiags());
}
+#endif
} // namespace
} // namespace clangd
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
