<table border="1" cellspacing="0" cellpadding="8">
    <tr>
        <th>Issue</th>
        <td>
            <a href=https://github.com/llvm/llvm-project/issues/99612>99612</a>
        </td>
    </tr>

    <tr>
        <th>Summary</th>
        <td>
            [C++20] [Modules] Ambiguousness of owning modules
        </td>
    </tr>

    <tr>
      <th>Labels</th>
      <td>
            metabug,
            clang:modules
      </td>
    </tr>

    <tr>
      <th>Assignees</th>
      <td>
      </td>
    </tr>

    <tr>
      <th>Reporter</th>
      <td>
          ChuanqiXu9
      </td>
    </tr>
</table>

<pre>
    I try to summarize the concern about the compatibilities of named modules and PCH and header like modules (clang modules and header units). I try to use this page to describe the meta issue even if there is no such reports now. 

### Background

In short, due to named modules bring new semantics, we need to change the following questions in a lot of places:
- If a declaration belongs to a **global module fragment**.
- If a declaration belongs to a **global module fragment** of other translation units than the current one we're handling.
- If a declaration belongs to a named module. If yes, we need to get its name. And if it is the one we're parsing.

For the first question, now we use `D->getOwningModule()->isExplicitGlobalModule()` to get the answer. It works fine. But if we meet PCH and header like modules. It just doesn't work. For example,

```
module;
#include "header.h" // replaced by clang header modules
export module m;
...
```

In this example, if `#include "header.h"` got replaced by `clang header modules`, the returned module will be the clang header module instead of the global module fragment we want. Then, mismatch happens.

I think the key point here is that we abused the `clang::Module` class. I think, **ideally, there will be two kinds of modules, one is in the serialization level like `clang::Module` and another one should be used at semantical level**.  But we don't have that one now and we're using `clang::Module` to do that. If we had that, we can always use the semantical level modules for questions I raised.

So it looks like we probably have to do some large scale refactors.

### Some ideas

>From the perspective of the specification, there are only 2 kinds of modules in C++: the global module and named modules. Then I **thought** if we can treat all header like modules as the global module to workaround? It is fine for header units but not for clang modules (or it is not well defined).

For example,

```
export module a;
#include "my_headers" // replaced by clang header modules
```

If `#include "my_headers"` got replaced by clang header modules, then should the declarations in `my_headers` be treated as if in named modules? Or should we treat them as in global module just like header units?

### PCH

PCH looks more odd with named modules. For example, for

```
module;
#include "header.h"
export module m;
...
```

if we `include-pch` for `header.h`, as far I understood, it is like doing:

```
#include-pch "header.h"
module;
#include "header.h"
export module m;
...
```

But ... it is invalid to include things before `module;`! So it looks more weird to me. And I don't know what happens if we write the module unit with PCH included precisely, I feel it is a dark corner.

---

Maybe there are other problems. And I don't have a very clear idea for the refactoring I said above. But it should be good to make such dark corners more explicit and track them in one place.

CC: @dwblaikie @mizvekov @zygoloid @Bigcheese @jansvoboda11 @vgvassilev 


</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJy8V0Fv6zgO_jXKhajhKE3aHHJo86Y7OTzMALOHvS1oi7H1IksZSY5f3q9fUHLSuJMuZrGDAYI2jiXyI_nxo4Qh6MYSbcTyVSy_zLCPrfObbduj_V3_q1_PKqfOmx1Ef4boIPRdh17_IIgtQe1sTd4CVq6P4y_dEaOutNFRUwC3B4sdKeic6g0FQKvg1-3P6X9LqMiD0Qe6vhfyuTZom8mGcWFvdQxCrgu44ukDI9EBjtgQ_6Ao1F5XGV9HEUGH0BPQiSzoPf_sCXQAy8HULXg6Oh_5eShAlF9E-TL-lYv8gVesD413vVW373cWQut8FHILqk_Op6FWXtsGLA0QqEMbdR147UBgiRSvr1u0TYa6d8a4gTf83lOI2tkA2gKCcZGzeDRYUxCL0fkD7PaAoKg26JGXQ0XG2SawXQQhX4R8aYyr0IyAYO-x6cjG_K74qwwxOsdphejRBpONpFpBbNFmXvTek43gLMFAQj55ghatMto2fxLJbXILXnumj_lsKAK75aUFvFjFFdeRy80gbp0f0Yd33_nvm_O5FtqHeK0D-7BuYD_MNrEqvzyIxU8NxV8Gq23zNSES8lnINb_Q4afvR6NrHf-Rsnb7XqzKC052hDYM5AvYRRicPwTYa0sFvPaRgQ9MYIr_rV3S1m99iKAcBSvkU7ZUAMdC37E7suvthNercvykx2xJLF6vrNe2Nr0iEFJml0UrpAQh34R8435hKiqozpA7dcQ1Qspm6Dt31YUw3dV8URR3UVxbKjXzO3DOAy_6DBUntHFxgkqsyrvA2Mw25d1T7L29sgkGbQyMmnFnK2gbIqFipvOS-93A9RrQxgL-2VJiTadDh7FuocXjkWyYkG3HodpDMnigMxydthEu6hRbTAax6gNzu6VrWKwBi5eRVauSAYeQJJHtsd_cl1oRGnMeY_Y3UQ4ODtqqpM6X5Mhtag-dVIfdBfIajf6RO9HQiUym3mc4mKJosxKwqdC63ih2mELAeJVBNNneKESQCD8QKJcZ3OKJcgbYDvce2750bs99-ykKHgEubU4aMbDMqPQ8akWNFtAMeA7j8KA_4Loq-N75GznegUcdSE3K-JtjgTHOHUJOz0Bw9K7CypzHQBKi4DoCg74hCDUa5uAe6-j8lBXvQ-c33sA1DBOJ8q5LmI_kw5HqqE90oSU_672u8aJauezoWfnMGeQfqs613gr5yp_Fyx1uc94nQy2TG3Yjx2Lr-qa9DIKsWZzg6AkjoDF3ZzyGO66iS8qFec4u3ljZdFbEVIfbMwBUfQTrYnoxPS4I-ez8qPm8YiBjQBGbUXx0-Kj3f0ojp2qGn4hld_53Bhn-Z7m8r4Z3pG_i45743fWRuWAvLcnJvxmziQZiVd7YXpVJKLiK3LohzVE7pQLX6Bd_sTmMy9l4l3bYDwVOYyrxYHKcW7zd5_-v259vX_AQzF3WOSa0UjDo2H5k57SkzI-_YPT9PyMt94RYlaPph2PdcnqZuWJVXr3k4YQB9uhhB73lOkTnVBqCic4pd8rpLHqfh_UeBzu7G8vfETmLelEUI3htT2h0OqVdXPG8agJUtOeKMgGvqDgbc7jV1lT1gbRPJi7Hu911ZhzSEY1nxjhsRzUavI7jZSDDZ9Zl6jClRiwKjp5qHSjPyx3sicyIHEGhP0DtvCU_0Y-Hh4fbx694zmeIi-imUcjDwFAXPgJOswHhRJ6bltAnsU-8yEeUPB541u0goFZ8yzpdTofxZrw2zuWk4IHyneYG8Jg5Go-kSdGjx_qQG1XbNGOTfkxi2255IojHUg2VQX3QxA-d_nGigzvx9x_nxhmnFX9_1U3dEoW06BvacHKVUzif8_OpOfE109Bpcr-aqc1CrRdrnNFm_iTni6fVaj2ftRu5eq7n9dP8-VGul_On_WK5Wj0vnxaLlVrV--VipjeylI_l03xdPs_ni3mh5vPlUiq1lFX5WEkpHkvqUJvCmFNXON_M0iVws16v5nJmsCIT0o1XSr4iVn3DXJdbIeXlUHGVTsnXYr9hSw9V3wTxWBodYni3HXU06QI9jlJZiuUXEMvXr6ON5Rd46Srd9K4PlkKawC7dHS7CNeu92bQxHtMNL02ORse2r4radUK-sa_x38PRu29URyHfUlBByLcc12kj_xMAAP__QUAeYA">