[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