[PATCH] D123837: [C++20] [Modules] Judge isInCurrentModule currently

Chuanqi Xu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 19 03:29:42 PDT 2022


ChuanqiXu added inline comments.


================
Comment at: clang/lib/Sema/SemaLookup.cpp:1579
+      (M->getPrimaryModuleInterfaceName() ==
+       llvm::StringRef(getLangOpts().CurrentModule).split(':').first)) {
+    CurrentModuleUnitsCache.insert(M);
----------------
I thought to add a comment for this. But I feel like it might be too wordy...


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D123837/new/

https://reviews.llvm.org/D123837



More information about the cfe-commits mailing list