[PassManager] Drop Function.h and Module.h includes (NFC)

[nikic]

Replace them with forward declarations.

[llvmbot]

@llvm/pr-subscribers-llvm-ir

Author: Nikita Popov (nikic)

Changes

Replace them with forward declarations.

---

Full diff: https://github.com/llvm/llvm-project/pull/96892.diff

1 Files Affected:

• (modified) llvm/include/llvm/IR/PassManager.h (+3-2)

diff --git a/llvm/include/llvm/IR/PassManager.h b/llvm/include/llvm/IR/PassManager.h
index 226eba7d8027a..d269221fac070 100644
--- a/llvm/include/llvm/IR/PassManager.h
+++ b/llvm/include/llvm/IR/PassManager.h
@@ -42,8 +42,6 @@
 #include "llvm/ADT/StringRef.h"
 #include "llvm/ADT/TinyPtrVector.h"
 #include "llvm/IR/Analysis.h"
-#include "llvm/IR/Function.h"
-#include "llvm/IR/Module.h"
 #include "llvm/IR/PassManagerInternal.h"
 #include "llvm/Support/TypeName.h"
 #include <cassert>
@@ -58,6 +56,9 @@
 
 namespace llvm {
 
+class Function;
+class Module;
+
 // Forward declare the analysis manager template.
 template <typename IRUnitT, typename... ExtraArgTs> class AnalysisManager;
 

[nikic]

Turns out this doesn't really do anything, because the headers still get pulled in by Analysis.h, and dropping them from there is significantly more involved. Still going to merge this to clarify that this header doesn't need them.