[clang] 1c3df53 - [NFC] Explicitly use `clang::Module` in test file
via cfe-commits
cfe-commits at lists.llvm.org
Wed Jan 15 13:44:30 PST 2025
Author: Hubert Tong
Date: 2025-01-15T17:44:25-04:00
New Revision: 1c3df5308a86aaed4c91869982d365cc20f2b511
URL: https://github.com/llvm/llvm-project/commit/1c3df5308a86aaed4c91869982d365cc20f2b511
DIFF: https://github.com/llvm/llvm-project/commit/1c3df5308a86aaed4c91869982d365cc20f2b511.diff
LOG: [NFC] Explicitly use `clang::Module` in test file
https://github.com/llvm/llvm-project/pull/122887 uses `Module` to refer
to `clang::Module` in a test that has `using namespace llvm;`.
This causes lookup ambiguity with `llvm::Module` if the headers involved
expose that name (e.g., for downstream codebases).
Added:
Modified:
clang/unittests/AST/ExternalASTSourceTest.cpp
Removed:
################################################################################
diff --git a/clang/unittests/AST/ExternalASTSourceTest.cpp b/clang/unittests/AST/ExternalASTSourceTest.cpp
index ad209604971f43..b42a3be05e9bb1 100644
--- a/clang/unittests/AST/ExternalASTSourceTest.cpp
+++ b/clang/unittests/AST/ExternalASTSourceTest.cpp
@@ -69,7 +69,7 @@ TEST(ExternalASTSourceTest, FailedLookupOccursOnce) {
bool FindExternalVisibleDeclsByName(const DeclContext *,
DeclarationName Name,
- Module *NamedModule) override {
+ clang::Module *NamedModule) override {
if (Name.getAsString() == "j")
++Calls;
return false;
More information about the cfe-commits
mailing list