[clang-tools-extra] [clangd] [C++ Modules] Skip PCH when TU imports modules (PR #187432)

[Aaron Ballman via cfe-commits]

AaronBallman wrote:

> I think, to move on, I'd like to take the maintainer ship of modules in clangd. I think I can make the right call. I think, in this community, I am the one who cares about C++20 modules users the most. I think we need to care about users in reality instead of in imagination.

I don't think we should make a distinction like that -- modules are a clang feature, clangd is effectively a downstream consumer of clang. So in terms of *modules* functionality, I think you (and the other modules/serialization maintainers) are on the hook for decisions. In terms of *clangd* functionality, I think that's up to the clandg maintainers. And when there's sustained disagreement between folks, we should schedule a call to try to resolve the disagreements (through the @llvm/clang-area-team). Let me know if you think we're to that point already, I can put this on the area team's agenda to schedule a call for it.

https://github.com/llvm/llvm-project/pull/187432