Skip to content

[flang][cuda] Add missing dependency #106298

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 28, 2024
Merged

Conversation

clementval
Copy link
Contributor

Add missing dependency that sometimes makes a build fails with ninja.

@clementval clementval requested a review from wangzpgi August 27, 2024 21:57
@llvmbot llvmbot added flang Flang issues not falling into any other category flang:fir-hlfir labels Aug 27, 2024
@llvmbot
Copy link
Member

llvmbot commented Aug 27, 2024

@llvm/pr-subscribers-flang-fir-hlfir

Author: Valentin Clement (バレンタイン クレメン) (clementval)

Changes

Add missing dependency that sometimes makes a build fails with ninja.


Full diff: https://github.com/llvm/llvm-project/pull/106298.diff

1 Files Affected:

  • (modified) flang/lib/Optimizer/Dialect/CUF/Attributes/CMakeLists.txt (+1)
diff --git a/flang/lib/Optimizer/Dialect/CUF/Attributes/CMakeLists.txt b/flang/lib/Optimizer/Dialect/CUF/Attributes/CMakeLists.txt
index 81db40f3ba466d..ec5484c1d61080 100644
--- a/flang/lib/Optimizer/Dialect/CUF/Attributes/CMakeLists.txt
+++ b/flang/lib/Optimizer/Dialect/CUF/Attributes/CMakeLists.txt
@@ -5,6 +5,7 @@ add_flang_library(CUFAttrs
   DEPENDS
   MLIRIR
   CUFAttrsIncGen
+  CUFOpsIncGen
 
   LINK_LIBS
   MLIRTargetLLVMIRExport

@clementval clementval merged commit f215447 into llvm:main Aug 28, 2024
11 checks passed
@clementval clementval deleted the cuf_deps branch August 28, 2024 00:36
dmpolukhin pushed a commit to dmpolukhin/llvm-project that referenced this pull request Sep 2, 2024
Add missing dependency that sometimes makes a build fails with ninja.
dmpolukhin pushed a commit to dmpolukhin/llvm-project that referenced this pull request Sep 6, 2024
Add missing dependency that sometimes makes a build fails with ninja.
dmasloff pushed a commit to dmasloff/llvm-project-copy that referenced this pull request Jan 1, 2025
Add missing dependency that sometimes makes a build fails with ninja.
dmasloff pushed a commit to dmasloff/llvm-project-copy that referenced this pull request Jan 2, 2025
Add missing dependency that sometimes makes a build fails with ninja.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flang:fir-hlfir flang Flang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants