[PATCH] D145965: [C++20][Modules] Fix incorrect visibilities in implementation units.

Tue Mar 14 02:33:57 PDT 2023

iains added a comment.

In D145965#4192072 <https://reviews.llvm.org/D145965#4192072>, @ChuanqiXu wrote:

> In D145965#4192051 <https://reviews.llvm.org/D145965#4192051>, @iains wrote:
>
>> 

>>
>>
>>
>> The checks for internal-linkage symbols and the improvements to diagnostics do not depend on **how** `Sema::isModuleUnitOfCurrentTU` is implemented, so that I think we could fix these problems and then deal with the refactoring later.
>>
>> In either case, I do not have much time to work more on this right now.
>
> I'll try to make it.

Perhaps then, we can split out the changes to `Sema::isModuleUnitOfCurrentTU` and see how much of the functionality is still working (with the intent that the full fix will come along  later).

So if this is split into two

1. the fix to `Sema::isModuleUnitOfCurrentTU`  (which you will be updating) and
2. the uses of that to fix the lookups and diagnostics?

(I can use both as a temporary fix ..  but 2 could be applied with the existing `isModuleUnitOfCurrentTU` implementation).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145965