update toolchain to gcc13.2 #9579
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
stronnag
commented
Dec 19, 2023
stronnag
commented
- rework for Arm's new naming / paths / case inconsistency
- support native binary on darwin/aarch64
- fix ATF4 link warning
- workaround spurious linker warnings for unused functions
add "-Wl,--no-warn-rwx-segments" for ATF4
|
As will be seen from the cmake file, there is a fairly ugly workaround for some strange behaviour in newlib 4.3, which results in many (16) link warnings for functions not used (AFAIK) by INAV:
In order to suppress the warnings, as they (a) appear spurious / harmless and (b) would be a support burden, the warnings are suppressed by removing the It is possible that someone with rather more insight into the Arm compiler / linker may have a better solution. With the above workaround, all release targets build locally without warnings. |
Comparative images sizes for various firmwareGcc 10.3Gcc 13.2.1 |
|
All release firmware link sizes (no warnings present) |
|
Test Built on the following platforms:
Not tested on:
|
DzikuVx
approved these changes
Dec 20, 2023
JulianTiller
pushed a commit
to JulianTiller/inav
that referenced
this pull request
Oct 18, 2024
add "-Wl,--no-warn-rwx-segments" for ATF4
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.