================
@@ -42,7 +41,11 @@
 #include <memory>
 #include <vector>
 
-namespace clang::clangd {
+namespace clang {
+
+class Module;
----------------
kadircet wrote:

this is not an "unnecessary" include, clangd requires a declaration of 
`clang::Module` in this file (even if incomplete) and we try not to have 
forward declarations.

what's the motivation behind this change exactly?

https://github.com/llvm/llvm-project/pull/93417
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to