[llvm] llvm: Replace forward declaration with include (PR #171848)
Fabrice de Gans via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 11 07:25:46 PST 2025
https://github.com/Steelskin created https://github.com/llvm/llvm-project/pull/171848
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.
>From 19ab0e86bbf05bb2e51edc490e003002b94475da Mon Sep 17 00:00:00 2001
From: Fabroce de Gans <fabrice at thebrowser.company>
Date: Thu, 11 Dec 2025 16:23:57 +0100
Subject: [PATCH] llvm: Replace forward declaration with include
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.
---
.../llvm/Transforms/Instrumentation/SanitizerBinaryMetadata.h | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
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;
More information about the llvm-commits
mailing list