[PATCH] D118589: [C++20][Modules][6/8] Record direct module imports.
Iain Sandoe via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 17 07:18:44 PST 2022
iains updated this revision to Diff 409645.
iains added a comment.
rebased onto parent patch changes
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D118589/new/
https://reviews.llvm.org/D118589
Files:
clang/include/clang/Sema/Sema.h
clang/lib/Sema/SemaModule.cpp
Index: clang/lib/Sema/SemaModule.cpp
===================================================================
--- clang/lib/Sema/SemaModule.cpp
+++ clang/lib/Sema/SemaModule.cpp
@@ -513,6 +513,11 @@
assert(ThisModule && "was expecting a module if building one");
}
+ // In some cases we need to know if an entity was present in a directly-
+ // imported module (as opposed to a transitive import). This avoids
+ // searching both Imports and Exports.
+ DirectModuleImports.insert(Mod);
+
return Import;
}
Index: clang/include/clang/Sema/Sema.h
===================================================================
--- clang/include/clang/Sema/Sema.h
+++ clang/include/clang/Sema/Sema.h
@@ -2222,6 +2222,9 @@
/// The global module fragment of the current translation unit.
clang::Module *GlobalModuleFragment = nullptr;
+ /// The modules we imported directly.
+ llvm::SmallPtrSet<clang::Module *, 8> DirectModuleImports;
+
/// Namespace definitions that we will export when they finish.
llvm::SmallPtrSet<const NamespaceDecl*, 8> DeferredExportedNamespaces;
@@ -2249,6 +2252,10 @@
return Entity->getOwningModule();
}
+ bool isModuleDirectlyImported(const Module *M) {
+ return DirectModuleImports.contains(M);
+ }
+
/// Make a merged definition of an existing hidden definition \p ND
/// visible at the specified location.
void makeMergedDefinitionVisible(NamedDecl *ND);
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D118589.409645.patch
Type: text/x-patch
Size: 1427 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220217/4c7522fa/attachment.bin>
More information about the cfe-commits
mailing list