build.yml: Use a newer build toolchain #2306
Merged
Conversation
This file contains hidden or 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
|
@tannewt let me know what direction you'd like to see this go in, now that we know there are failures due to binary size increases |
|
Let's close this for now. We can update along with CP 6. The Pirkey is hundreds of bytes over which will be hard to recover. |
|
@tannewt makes sense, couldn't have been predicted without the CI run... |
|
I'm looking at this again, there are some optimization flags we can set so that gcc9 behaves closer to gcc7 in terms of code size. I think this may let the most constrained m0 boards fit again. |
.. the new version is not offered in .deb form, so the installation process also changes
tannewt
requested changes
Nov 26, 2019
By tweaking the optimizer flags, we can get back a few hundred bytes and allow these builds to work with the gcc9 compiler.
By tweaking the optimizer flags further, this build also fits
pewpew10, pirkey_m0, and uchip all fit now. However, pirkey_m0 now has just 76 bytes flash available.
|
Sigh another board that doesn't fit |
.. inline-unit-growth was the same across all boards, and the highest max-inline-insns-auto parameter was shared across 2 of 5 boards, so it's worth a little work to follow the DRY principle
tannewt
approved these changes
Dec 3, 2019
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.
.. the new version is not offered in .deb form, so the installation process also changes
Problems: