[PATCH] D153542: [C++20][Modules] Implement P2615R1 exported specialization diagnostics.

Iain Sandoe via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 26 06:36:51 PDT 2023


iains added inline comments.


================
Comment at: clang/lib/Sema/SemaModule.cpp:848-849
+    if (auto *FD = dyn_cast<FunctionDecl>(D)) {
+      if (FD->getTemplateSpecializationKind() == TSK_ExplicitSpecialization)
+        BadExport = true;
+    } else if (auto *VD = dyn_cast<VarDecl>(D)) {
----------------
ChuanqiXu wrote:
> Given P2615R1 doesn't allow explicit-instantiation in export block too.
I think I must be missing something - I do not see that in the paper - please could you expand your comment?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153542



More information about the cfe-commits mailing list