[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:50:16 PST 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-analysis
Author: None (h-vetinari)
<details>
<summary>Changes</summary>
`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
---
Full diff: https://github.com/llvm/llvm-project/pull/120661.diff
1 Files Affected:
- (modified) llvm/lib/Analysis/CMakeLists.txt (+1)
``````````diff
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
``````````
</details>
https://github.com/llvm/llvm-project/pull/120661
More information about the llvm-commits
mailing list