Skip to content

Conversation

sahas3
Copy link
Member

@sahas3 sahas3 commented Sep 20, 2025

@sahas3 sahas3 requested a review from saienduri September 20, 2025 23:08
@sahas3
Copy link
Member Author

sahas3 commented Sep 20, 2025

@saienduri I am not much familiar with how the bazel build is setup in torch-mlir. I couldn't figure out how to fix the other bazel failure in https://github.com/llvm/torch-mlir/actions/runs/17758450172/job/50465645001#step:8:2688 -- the file "mlir/Dialect/Func/Extensions/InlinerExtension.h" seems to exist in the llvm directory. Commenting out the include and the corresponding inliner registration call leads to a successful build but 12 tests out of 98 tests fails.

@saienduri
Copy link
Collaborator

Yeah that's fine. Just wanted to make sure the failure introduced due to #4291 is fixed.

@saienduri saienduri merged commit ed166d4 into llvm:main Sep 21, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants