make: build with linker garbage collection#3163
Conversation
|
Finally those cortexm clean-up PRs pay off :-) |
|
Errr what.. I can't remember doing that. Sorry. |
|
isn't it also necessary to add |
|
Ah never mind, seems like those options are enabled in |
|
@gebart Thanks for adopting this. Added the CI label and kicked Travis. |
|
|
|
ACK from my side and Travis is happy, should we merge this? |
|
Has someone tested? |
|
This should be fairly low-risk when building without LTO. With LTO enabled stuff might start disappearing if we forget to add |
|
Results from the examples directory: |
|
|
Currently only has the linker flag --gc-sections defined which tells the linker to omit dead code. Depending on the application this leads to significant reduction of code size.
|
@thomaseichinger LTO builds are broken on many platforms already... I'll add a warning for LTO in general |
dd72791 to
a624412
Compare
|
Rebased, opened #3376 for the warning |
|
ACK when Travis passed (again) and #3376 is merged. |
|
Why is Strider taking longer than Travis to test this? |
make: build with linker garbage collection
Adopted #1784 and rebased on latest master.
Currently only has the linker flag --gc-sections defined which tells
the linker to omit dead code. Depending on the application this leads
to significant reduction of code size.