[llvm] [llvm] add vt_gen target as a dependency of llvm/lib/Analysis (PR #120661)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 19 16:49:39 PST 2024
https://github.com/h-vetinari created https://github.com/llvm/llvm-project/pull/120661
`DroppedVariableStats.cpp` includes `llvm/include/llvm/Analysis/DroppedVariableStats.h` through its header, which transitively requires `llvm/CodeGen/GenVT.inc` (which is what `vt_gen` depends on) through `llvm/include/llvm/CodeGenTypes/MachineValueType.h`
Based on my testing, this should address #107687, at least for the case where only llvm is being built. There may be further places in clang, mlir etc. that need to add `DEPENDS vt_gen` for using `llvm/include/llvm/CodeGenTypes/MachineValueType.h`
CC @chapuni as author of https://github.com/llvm/llvm-project/commit/631bfdbee5b45eda9f99dff6a716d63c5698e4bd
>From 6443db33cee640465d1c9645264e32ff024403f2 Mon Sep 17 00:00:00 2001
From: "H. Vetinari" <h.vetinari at gmx.com>
Date: Fri, 20 Dec 2024 11:07:01 +1100
Subject: [PATCH] [llvm] add vt_gen target as a dependency of llvm/lib/Analysis
DroppedVariableStats.cpp includes `llvm/include/llvm/Analysis/DroppedVariableStats.h`
through its header, which transitively requires `llvm/CodeGen/GenVT.inc` (which is
what `vt_gen` depends on) through `llvm/include/llvm/CodeGenTypes/MachineValueType.h`
---
llvm/lib/Analysis/CMakeLists.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/llvm/lib/Analysis/CMakeLists.txt b/llvm/lib/Analysis/CMakeLists.txt
index 0db5b80f336cb5..cf393850ec3e68 100644
--- a/llvm/lib/Analysis/CMakeLists.txt
+++ b/llvm/lib/Analysis/CMakeLists.txt
@@ -150,6 +150,7 @@ add_llvm_component_library(LLVMAnalysis
DEPENDS
intrinsics_gen
+ vt_gen
${MLDeps}
LINK_LIBS
More information about the llvm-commits
mailing list