[clang] [clang-tools-extra] [libcxx] [clang][Modules] Remove unnecessary includes of `Module.h` (PR #93417)
kadir çetinkaya via cfe-commits
cfe-commits at lists.llvm.org
Mon May 27 09:08:49 PDT 2024
================
@@ -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
More information about the cfe-commits
mailing list