[clang] [clang][test] Add missing test file to cmake (PR #99907)

via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 22 10:34:01 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Keith Smiley (keith)

<details>
<summary>Changes</summary>

Seems like this test was never running with cmake, but is running with
bazel and broke at head.


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


2 Files Affected:

- (modified) clang/unittests/Tooling/CMakeLists.txt (+1) 
- (modified) clang/unittests/Tooling/RecursiveASTVisitorTests/DeductionGuide.cpp (+1-1) 


``````````diff
diff --git a/clang/unittests/Tooling/CMakeLists.txt b/clang/unittests/Tooling/CMakeLists.txt
index 0eb612f8d9498..401978c31863c 100644
--- a/clang/unittests/Tooling/CMakeLists.txt
+++ b/clang/unittests/Tooling/CMakeLists.txt
@@ -40,6 +40,7 @@ add_clang_unittest(ToolingTests
   RecursiveASTVisitorTests/CXXMethodDecl.cpp
   RecursiveASTVisitorTests/CXXOperatorCallExprTraverser.cpp
   RecursiveASTVisitorTests/DeclRefExpr.cpp
+  RecursiveASTVisitorTests/DeductionGuide.cpp
   RecursiveASTVisitorTests/ImplicitCtor.cpp
   RecursiveASTVisitorTests/ImplicitCtorInitializer.cpp
   RecursiveASTVisitorTests/InitListExprPostOrder.cpp
diff --git a/clang/unittests/Tooling/RecursiveASTVisitorTests/DeductionGuide.cpp b/clang/unittests/Tooling/RecursiveASTVisitorTests/DeductionGuide.cpp
index cd4bf0eb7bd5a..27cb39ea8fdf7 100644
--- a/clang/unittests/Tooling/RecursiveASTVisitorTests/DeductionGuide.cpp
+++ b/clang/unittests/Tooling/RecursiveASTVisitorTests/DeductionGuide.cpp
@@ -58,7 +58,7 @@ Bar s(1);
       DeductionGuideVisitor::Lang_CXX2a));
 }
 
-TEST(RecursiveASTVisitor, DeductionGuideImplicitMode) {
+TEST(RecursiveASTVisitor, DISABLED_DeductionGuideImplicitMode) {
   DeductionGuideVisitor Visitor(/*ShouldVisitImplicitCode*/ true);
   Visitor.ExpectMatch("Foo(T) -> Foo<int>", 11, 1);
   Visitor.ExpectMatch("Bar(type-parameter-0-0) -> Foo<int>", 14, 1);

``````````

</details>


https://github.com/llvm/llvm-project/pull/99907


More information about the cfe-commits mailing list