Skip to content

Use new linking support #31

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

Merged
merged 9 commits into from
Mar 21, 2025
Merged

Use new linking support #31

merged 9 commits into from
Mar 21, 2025

Conversation

jonnor
Copy link
Contributor

@jonnor jonnor commented Mar 21, 2025

No description provided.

@jonnor
Copy link
Contributor Author

jonnor commented Mar 21, 2025

Tested working on ESP32-S3 with MicroPython 1.24.1, so that seems good.
However on Gitlab CI the modules fail to build on armv6m, with unsupported relocation.

On building emlearn_trees

   File "/home/runner/work/emlearn-micropython/emlearn-micropython/micropython/tools/mpy_ld.py", line 707, in do_relocation_text
    assert 0, r_info_type
AssertionError: 2

This does not happen locally for me, with arm-none-eabi 14.1.0 on Arch Linux. Whereas CI has arm-none-eabi 13.2.1 on Ubuntu 24.04. So it might be version specific?

@jonnor
Copy link
Contributor Author

jonnor commented Mar 21, 2025

armv6m can be fixed later, as it is also broken in master (crashing).

@jonnor jonnor merged commit 251928c into master Mar 21, 2025
4 checks passed
@jonnor jonnor deleted the linking branch March 21, 2025 06:57
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.

1 participant