Skip to content

Bump eudsl-python-extras to 0.1.0.20260308.1929+09d24cd#2941

Merged
hunhoffe merged 1 commit into
Xilinx:mainfrom
erwei-xilinx:bump-eudsl-python-extras
Mar 9, 2026
Merged

Bump eudsl-python-extras to 0.1.0.20260308.1929+09d24cd#2941
hunhoffe merged 1 commit into
Xilinx:mainfrom
erwei-xilinx:bump-eudsl-python-extras

Conversation

@erwei-xilinx
Copy link
Copy Markdown
Collaborator

Summary

Background

Issue #2937 reported ImportError: cannot import name '_is_integer_like_type' when using the IRON API. Investigation showed the root cause was the old eudsl-python-extras (3c7ac1b, Dec 2025) importing helper functions that LLVM 23.0.0 removed from arith.py and emitter.py. The fix was already applied in #2918 by updating the eudsl pin to 549fd26, which uses Python isinstance() instead of helper function imports — compatible with LLVM 23.0.0's proper Python type hierarchy (F32Type → FloatType → Type).

This PR bumps to the latest eudsl release for forward compatibility.

Test plan

  • Reproduced original issue: old eudsl 3c7ac1b + LLVM 23.0.0 → ImportError: cannot import name '_is_integer_like_type'
  • Verified fix: new eudsl 09d24cd + LLVM 23.0.0 wheel → imports succeed
  • Ran vendor_eudsl.py with new version — vendored arith.py uses isinstance() (not removed helper functions)
  • e2e design tests pass: code_region.py (ObjectFifo MLIR gen), aie_ops.py (device configs), npu.py (matmul/edge_detect designs)
  • Type hierarchy verified: isinstance(F32Type, FloatType) and isinstance(MemRefType, ShapedType) both work

Closes #2937

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings March 9, 2026 00:23
Updates eudsl-python-extras from 0.1.0.20260211.1045+549fd26 to
0.1.0.20260308.1929+09d24cd. This picks up the latest upstream fixes
including the memref.reinterpret_cast mixed static/dynamic
offsets/sizes/strides fix (llvm/eudsl#355).

The new version continues to use Python isinstance() for type checks
(compatible with LLVM 23.0.0), ensuring no import errors as reported
in Xilinx#2937.

Closes Xilinx#2937

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the pinned eudsl-python-extras dependency version in the Python requirements to pick up upstream fixes and maintain compatibility with LLVM 23.0.0 Python bindings (notably avoiding removed helper-function imports).

Changes:

  • Bump eudsl-python-extras from 0.1.0.20260211.1045+549fd26 to 0.1.0.20260308.1929+09d24cd.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@erwei-xilinx
Copy link
Copy Markdown
Collaborator Author

CI Failure Analysis

Run Tests on Ryzen AI (amd7940hs)Unrelated crash in C++ aiecc

The failing test is aiecc/cpp_multi_device_sequence.mlir. The aiecc binary crashes during the "Running resource allocation pipeline in-memory" step (stack trace in the FileCheck dump: PLEASE submit a bug report to https://github.com/llvm/llvm-project/issues/). This is a crash in the C++ aiecc compiler driver, completely unrelated to the eudsl-python-extras Python version bump in this PR.

The same test passes on other recent PRs (#2940, #2939, #2936, #2935, #2933) and on main. This appears to be an intermittent crash in the aiecc binary on the amd7940hs runner.

windows-2022 msvc assert=ON rtti=ONCache infrastructure error

Restoring cache failed: Error: The process 'C:\Program Files\Git\bin\sh.exe' failed with exit code 9

This is a GitHub Actions cache restoration failure, not a build/test failure.

All other checks pass (40+ jobs)

  • All Build and Test jobs across Ubuntu 22.04/24.04, Python 3.10-3.14, Assert/Release ✅
  • All wheel builds (10 configs) ✅
  • Lint, format, coverage ✅
  • Ryzen AI examples + amdhx370 tests ✅
  • Vitis tests ✅

Copy link
Copy Markdown
Collaborator

@hunhoffe hunhoffe left a comment

Choose a reason for hiding this comment

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

Thanks Erwei!

@hunhoffe hunhoffe added this pull request to the merge queue Mar 9, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 9, 2026
@hunhoffe hunhoffe added this pull request to the merge queue Mar 9, 2026
Merged via the queue into Xilinx:main with commit 6cd2b65 Mar 9, 2026
86 of 88 checks passed
@erwei-xilinx erwei-xilinx deleted the bump-eudsl-python-extras branch March 9, 2026 16:17
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.

Python import errors in mlir-aie wheel 0.0.1.2026030604: Missing type checking functions

4 participants