-
Notifications
You must be signed in to change notification settings - Fork 3k
Turn on ARM linker --inline #12825
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
Turn on ARM linker --inline #12825
Conversation
For release and develop profiles, turn on the linker's `--inline` optimisation. This can save a couple of hundred bytes in a typical image. The optimisation replaces branches to small functions with the inlined code from those functions. And it's possible that the out-of-line functions can be eliminated. Setting confined to release and develop builds only, as it can lead to invalid debug information.
@kjbracey-arm, thank you for your changes. |
CI started |
Test run: FAILEDSummary: 1 of 6 test jobs failed Failed test jobs:
|
test restarted |
Test failures not related, will restart CI later today |
CI restarted |
Test run: SUCCESSSummary: 6 of 6 test jobs passed |
@ARMmbed/mbed-os-core @ARMmbed/mbed-os-tools Please review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Summary of changes
For release and develop profiles, turn on the linker's
--inline
optimisation. This can save a couple of hundred bytes in a typical
image.
The optimisation replaces branches to small functions with the
inlined code from those functions. And it's possible that the
out-of-line functions can be eliminated.
Setting confined to release and develop builds only, as it can lead to
invalid debug information.
Impact of changes
Migration actions required
Documentation
None
Pull request type
Test results
Reviewers