[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