[PATCH] D122119: [C++20][Modules] Adjust handling of exports of namespaces and using-decls.

Wed Mar 23 19:13:25 PDT 2022

ChuanqiXu accepted this revision.
ChuanqiXu added a comment.
This revision is now accepted and ready to land.

In D122119#3402168 <https://reviews.llvm.org/D122119#3402168>, @urnathan wrote:

> In D122119#3401322 <https://reviews.llvm.org/D122119#3401322>, @ChuanqiXu wrote:
>
>> 
>
>
>
>> I think the example is invalid since it violates [[ http://eel.is/c++draft/module.interface#6 | [module.interface]p6 ]] explicitly. If this is intended behavior or by design, we should change the wording.
>
> That wording failed to be updated when the linkage-decl changes went in.  I'll add a note to DR 2541. Linkage specifications, module purview, and module attachment

OK, if the wording is not consistent with the design, I have no further objections.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122119