Skip to content
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

When compiling with toolchain, ran out of registers #8

Closed
ItsSven64 opened this issue Dec 27, 2023 · 7 comments
Closed

When compiling with toolchain, ran out of registers #8

ItsSven64 opened this issue Dec 27, 2023 · 7 comments

Comments

@ItsSven64
Copy link

Title, i get this error
image

Do you know about this?

@ItsSven64
Copy link
Author

After some back and forth with the toolchain team we figured out that it has to do with compiler issues. I added LTO = NO to the makefile, and now it throws the following;
image

@MathisLav
Copy link
Owner

Well, I'm not sure I understand what's causing this error. Maybe you're not using the same version of the toolchain as I am? (but your CEdev path suggests that you are)

Anyway, this seems to be more of a compiler side issue, I don't think I can help you with that, sorry. You could still reinstall the toolchain with the lastest version, it solves most issues, from experience.

@ItsSven64
Copy link
Author

Thanks for the help! Already opened an issue in toolchain

@mateoconlechuga
Copy link

mateoconlechuga commented Jan 17, 2024

The issue is the transport_checksum function. Changing the uint16_t chksmmsb and uint16_t chksmlsb variables to be of type uint24_t appears to fix the compile issue.

Still a compiler issue, but at least it is fixable.

@MathisLav
Copy link
Owner

Yes I had this bad habit to always choose the smallest variable size I could use. But I'll release soon a new version that uses uint24_t whenever it's possible, so it should fix the issue.

Thanks for the clarification @mateoconlechuga

@MathisLav
Copy link
Owner

Should be fixed in the fix_recent_phones branch. @ItsSven64 if you see this message, could you try to compile this branch (git checkout fix_recent_phones) and tell me if you still have the issues you mentioned in this thread?

@MathisLav
Copy link
Owner

Closing the issue as I think the last merge solved it. But feel free to reopen it if you encounter a similar issue again.

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

No branches or pull requests

3 participants