Skip to content

Conversation

@makslevental
Copy link
Contributor

No description provided.

@llvmbot
Copy link
Member

llvmbot commented Jan 7, 2026

@llvm/pr-subscribers-mlir

Author: Maksim Levental (makslevental)

Changes

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

1 Files Affected:

  • (modified) mlir/lib/CAPI/Dialect/CMakeLists.txt (+1)
diff --git a/mlir/lib/CAPI/Dialect/CMakeLists.txt b/mlir/lib/CAPI/Dialect/CMakeLists.txt
index 83bc6ff38852c..d5adda6dc00ea 100644
--- a/mlir/lib/CAPI/Dialect/CMakeLists.txt
+++ b/mlir/lib/CAPI/Dialect/CMakeLists.txt
@@ -465,6 +465,7 @@ add_mlir_upstream_c_api_library(MLIRCAPITransformDialect
   LINK_LIBS PUBLIC
   MLIRCAPIIR
   MLIRTransformDialect
+  MLIRTransformDialectTransforms
 )
 
 add_mlir_upstream_c_api_library(MLIRCAPITransformDialectTransforms

@makslevental
Copy link
Contributor Author

makslevental commented Jan 7, 2026

@makslevental makslevental force-pushed the users/makslevental/fixcapitransform branch from b085a0c to 2245cd4 Compare January 7, 2026 21:37
@makslevental makslevental force-pushed the users/makslevental/fixcapitransform branch from 2245cd4 to 1ce47f8 Compare January 7, 2026 21:37
@makslevental makslevental enabled auto-merge (squash) January 7, 2026 21:45
@makslevental makslevental merged commit b646f3b into main Jan 7, 2026
8 of 9 checks passed
@makslevental makslevental deleted the users/makslevental/fixcapitransform branch January 7, 2026 21:47
@Kewen12
Copy link
Contributor

Kewen12 commented Jan 7, 2026

Thanks for the fix! It seems there is still a linker error for our bot. Would you mind taking a look? Thanks!

https://lab.llvm.org/buildbot/#/builders/203/builds/33367

@makslevental
Copy link
Contributor Author

Thanks for the fix! It seems there is still a linker error for our bot. Would you mind taking a look? Thanks!

https://lab.llvm.org/buildbot/#/builders/203/builds/33367

I would love to know how these are slipping through both pre-commit and various other bots... but okay let me do that one as well.

@makslevental
Copy link
Contributor Author

fix here #174876

@Kewen12
Copy link
Contributor

Kewen12 commented Jan 7, 2026

Thanks again for the quick fix!

kshitijvp pushed a commit to kshitijvp/llvm-project that referenced this pull request Jan 9, 2026
Priyanshu3820 pushed a commit to Priyanshu3820/llvm-project that referenced this pull request Jan 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants