Skip to content

Conversation

@silee2
Copy link
Contributor

@silee2 silee2 commented Aug 27, 2025

Add missing dependency MLIRSCFDialect from #154556

@llvmbot
Copy link
Member

llvmbot commented Aug 27, 2025

@llvm/pr-subscribers-mlir-gpu

Author: Sang Ik Lee (silee2)

Changes

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

1 Files Affected:

  • (modified) mlir/lib/Conversion/XeGPUToXeVM/CMakeLists.txt (+1)
diff --git a/mlir/lib/Conversion/XeGPUToXeVM/CMakeLists.txt b/mlir/lib/Conversion/XeGPUToXeVM/CMakeLists.txt
index ed54b0bb5ee81..6db306a33be86 100644
--- a/mlir/lib/Conversion/XeGPUToXeVM/CMakeLists.txt
+++ b/mlir/lib/Conversion/XeGPUToXeVM/CMakeLists.txt
@@ -19,6 +19,7 @@ add_mlir_conversion_library(MLIRXeGPUToXeVM
   MLIRVectorDialect
   MLIRArithDialect
   MLIRIndexDialect
+  MLIRSCFDialect
   MLIRXeGPUDialect
   MLIRPass
   MLIRTransforms

@llvmbot
Copy link
Member

llvmbot commented Aug 27, 2025

@llvm/pr-subscribers-mlir

Author: Sang Ik Lee (silee2)

Changes

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

1 Files Affected:

  • (modified) mlir/lib/Conversion/XeGPUToXeVM/CMakeLists.txt (+1)
diff --git a/mlir/lib/Conversion/XeGPUToXeVM/CMakeLists.txt b/mlir/lib/Conversion/XeGPUToXeVM/CMakeLists.txt
index ed54b0bb5ee81..6db306a33be86 100644
--- a/mlir/lib/Conversion/XeGPUToXeVM/CMakeLists.txt
+++ b/mlir/lib/Conversion/XeGPUToXeVM/CMakeLists.txt
@@ -19,6 +19,7 @@ add_mlir_conversion_library(MLIRXeGPUToXeVM
   MLIRVectorDialect
   MLIRArithDialect
   MLIRIndexDialect
+  MLIRSCFDialect
   MLIRXeGPUDialect
   MLIRPass
   MLIRTransforms

@silee2 silee2 enabled auto-merge (squash) August 27, 2025 23:42
@silee2 silee2 requested a review from charithaintc August 27, 2025 23:42
MLIRVectorDialect
MLIRArithDialect
MLIRIndexDialect
MLIRSCFDialect
Copy link
Contributor

Choose a reason for hiding this comment

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

This or MLIRSCFTransforms?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Both are needed.

@charithaintc
Copy link
Contributor

To be sure run CI with -DBUILD_SHARED_LIBS=ON locally.

@silee2 silee2 merged commit 9bf5bf3 into llvm:main Aug 27, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants