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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
