[llvm] llvm: Replace forward declaration with include (PR #171848)

[via llvm-commits]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-llvm-transforms

Author: Fabrice de Gans (Steelskin)

<details>
<summary>Changes</summary>

In `SanitizerBinaryMetadata`, the forward declaration for `vfs::FileSystem` is causing issues when building with `LLVM_BUILD_LLVM_DYLIB` to build LLVm as a DLL on Windows.

This effort is tracked in #<!-- -->109483.

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


1 Files Affected:

- (modified) llvm/include/llvm/Transforms/Instrumentation/SanitizerBinaryMetadata.h (+1-3) 


``````````diff
diff --git a/llvm/include/llvm/Transforms/Instrumentation/SanitizerBinaryMetadata.h b/llvm/include/llvm/Transforms/Instrumentation/SanitizerBinaryMetadata.h
index 63c5990a41741..09c8e16393e4c 100644
--- a/llvm/include/llvm/Transforms/Instrumentation/SanitizerBinaryMetadata.h
+++ b/llvm/include/llvm/Transforms/Instrumentation/SanitizerBinaryMetadata.h
@@ -18,12 +18,10 @@
 #include "llvm/IR/Module.h"
 #include "llvm/IR/PassManager.h"
 #include "llvm/Support/Compiler.h"
+#include "llvm/Support/VirtualFileSystem.h"
 #include "llvm/Transforms/Utils/Instrumentation.h"
 
 namespace llvm {
-namespace vfs {
-class FileSystem;
-} // namespace vfs
 
 struct SanitizerBinaryMetadataOptions {
   bool Covered = false;

``````````

</details>


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