[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