-
Notifications
You must be signed in to change notification settings - Fork 18
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
A recent update is causing a compile error: #17
Comments
I don't get the same error on my end, which version of gcc are you using? |
I wonder if this could be related to the change from -O3 to -O2 which was needed to fit in the new sample code and leave room for more features. The total size will be less now, but it requires that all the source files are recompiled before linking, otherwise the total size will be too large. Does it help if you force recompilation of all the sources? |
I also don't get a compile error, in @Jaffe- latest commit, or @circuitcreature new commit with midi cc. |
Im running gcc 13.2 on Mac. |
Did you manage to resolve this by compiling all sources again? (delete all .o files) |
I have not been able to get it to compile without changing it back to -O3. |
@circuitcreature do you still get compile errors when you build with make and -O2 opt. without PlatformIO? |
@beau-seidon I do. I can dig into this later. I might be able to dig up an older Mac to test on as well. Same error as before |
Ok, I'll dig into that as well to see what that function specifically does. Like, way down in there. My brother has a Mac now, I'll ask him to try and compile it on his lappy too to see if he gets the same error. |
https://stackoverflow.com/questions/8188849/avr-linker-error-relocation-truncated-to-fit |
Going to close this issue. |
Closed. avr-gcc was out of date.
|
There was a recent update in sample.c that is causing a compile error:
This line
The text was updated successfully, but these errors were encountered: