[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