[PATCH] D130864: [NFC] Introduce ASTContext::isInSameModule()
Chuanqi Xu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Jul 31 20:18:03 PDT 2022
ChuanqiXu created this revision.
ChuanqiXu added reviewers: iains, ilya-biryukov, erichkeane.
ChuanqiXu added a project: clang-modules.
Herald added a project: All.
ChuanqiXu requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Previously, when we want to compare if two module units are in the same module, we need to compare their primary module name, which is string comparing. String comparing is expensive and we want to avoid it. So here is the patch. This one should be a NFC patch.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D130864
Files:
clang/include/clang/AST/ASTContext.h
clang/include/clang/Sema/Sema.h
clang/lib/AST/ASTContext.cpp
clang/lib/Sema/SemaDecl.cpp
clang/lib/Sema/SemaLookup.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D130864.448908.patch
Type: text/x-patch
Size: 8009 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220801/26d3c44a/attachment.bin>
More information about the cfe-commits
mailing list