Skip to content

Conversation

@mgorny
Copy link
Member

@mgorny mgorny commented Oct 18, 2025

Regression introduced in #163543.

@llvmbot llvmbot added flang Flang issues not falling into any other category flang:fir-hlfir openacc labels Oct 18, 2025
@mgorny mgorny changed the title [flang] Fix OpenACCTransforms linking to liBMLIR [flang] Fix OpenACCTransforms linking to libMLIR Oct 18, 2025
Regression introduced in llvm#163543.

Signed-off-by: Michał Górny <[email protected]>
@mgorny mgorny force-pushed the flang-openacc-link branch from 5f5bc0f to 5ed9e2c Compare October 18, 2025 09:22
@llvmbot
Copy link
Member

llvmbot commented Oct 18, 2025

@llvm/pr-subscribers-openacc

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

Author: Michał Górny (mgorny)

Changes

Regression introduced in #163543.


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

1 Files Affected:

  • (modified) flang/lib/Optimizer/OpenACC/Transforms/CMakeLists.txt (+3-1)
diff --git a/flang/lib/Optimizer/OpenACC/Transforms/CMakeLists.txt b/flang/lib/Optimizer/OpenACC/Transforms/CMakeLists.txt
index 2427da03e1a3c..ed177baf52bea 100644
--- a/flang/lib/Optimizer/OpenACC/Transforms/CMakeLists.txt
+++ b/flang/lib/Optimizer/OpenACC/Transforms/CMakeLists.txt
@@ -5,8 +5,10 @@ add_flang_library(FIROpenACCTransforms
   FIROpenACCPassesIncGen
 
   LINK_LIBS
+  FIRDialect
+
+  MLIR_LIBS
   MLIRIR
   MLIRPass
-  FIRDialect
   MLIROpenACCDialect
 )

Copy link
Contributor

@razvanlupusoru razvanlupusoru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@mgorny mgorny merged commit 4b7b728 into llvm:main Oct 19, 2025
10 checks passed
@mgorny mgorny deleted the flang-openacc-link branch October 19, 2025 07:20
@mgorny
Copy link
Member Author

mgorny commented Oct 19, 2025

Thanks!

aokblast pushed a commit to aokblast/llvm-project that referenced this pull request Oct 30, 2025
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 openacc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants