Skip to content

Conversation

@makslevental
Copy link
Collaborator

@makslevental
Copy link
Collaborator Author

cc @banach-space @rolfmorel

@makslevental makslevental requested a review from marbre November 21, 2025 12:26
@rolfmorel
Copy link

rolfmorel commented Nov 21, 2025

Thanks @makslevental!

llvm/lighthouse#14 demonstrates a general compatibility issue with the mlir-python-bindings package (i.e. not lighthouse related).

EDIT: will build the torch_mlir package from this branch later today and report back if it resolves the problem.

@rolfmorel
Copy link

Great -- that resolves it! Confirmation that this fix indeed works:

$ build_tools/python_deploy/build_linux_packages.sh
...
$ uv venv -p 3.12
...
$ uv pip install -f https://llvm.github.io/eudsl mlir-python-bindings                                            
...
 + mlir-python-bindings==20251121+f9696949c
$ uv pip install build_tools/python_deploy/wheelhouse/torch_mlir-0.0.1-cp312-cp312-linux_x86_64.whl  # Couldn't be bothered to specify the version...
...
$ uv pip install torch
...
$ uv run python  # execute the minimal reproducer from llvm/lighthouse#14:
>>> from mlir import ir
>>> from torch_mlir  import fx
>>> print(ir.Context)
<class 'mlir._mlir_libs._site_initialize.<locals>.Context'>

If this could go in soon, that would be much appreciated! As it stands, this issue is blocking bumping torch-mlir on lighthouse (and some PRs there, e.g. llvm/lighthouse#22).

CC @banach-space @rengolin

@makslevental makslevental merged commit 1cf2871 into main Nov 21, 2025
3 checks passed
@makslevental makslevental deleted the users/makslevental/fixnbdomain branch November 21, 2025 23:26
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.

3 participants