-
Notifications
You must be signed in to change notification settings - Fork 14.3k
adding missing lib MLIRDestinationStyleOpInterface #115703
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
Conversation
@llvm/pr-subscribers-mlir Author: Frank Schlimbach (fschlimb) Changesfixing CI failures caused by #114238 by adding MLIRDestinationStyleOpInterface lib Full diff: https://github.com/llvm/llvm-project/pull/115703.diff 1 Files Affected:
diff --git a/mlir/lib/Dialect/Mesh/IR/CMakeLists.txt b/mlir/lib/Dialect/Mesh/IR/CMakeLists.txt
index 45ac9edb280bc9..3fea4d67430e08 100644
--- a/mlir/lib/Dialect/Mesh/IR/CMakeLists.txt
+++ b/mlir/lib/Dialect/Mesh/IR/CMakeLists.txt
@@ -13,4 +13,5 @@ add_mlir_dialect_library(MLIRMeshDialect
MLIRIR
MLIRSupport
MLIRViewLikeInterface
+ MLIRDestinationStyleOpInterface
)
|
Thank you for putting this up. I'll give it a quick run-through in a test. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
fixes the error seen on our buildbots in a local test.
@jplehr Thanks for your help! |
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/52/builds/3636 Here is the relevant piece of the build log for the reference
|
fixing CI failures caused by llvm#114238 by adding MLIRDestinationStyleOpInterface lib @jplehr @mfrancio @rengolin
fixing CI failures caused by #114238 by adding MLIRDestinationStyleOpInterface lib
@jplehr @mfrancio @rengolin