Skip to content

Do not warn if start of section is changed #484

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

Merged
merged 1 commit into from
Mar 27, 2019

Conversation

fpistm
Copy link
Member

@fpistm fpistm commented Mar 27, 2019

Remove ld option 'warn-section-align', as by default, SECTIONS command
does not specify a start address for the section, let ld do the work.
If a start address is specified no warn was displayed.

--warn-section-align
Warn if the address of an output section is changed because of
alignment. Typically, the alignment will be set by an input section.
The address will only be changed if it not explicitly specified;
that is, if the SECTIONS command does not specify a start address
for the section.

Fix #465

Remove ld option 'warn-section-align', as by default, SECTIONS command
does not specify a start address for the section, let ld do the work.
If a start address is specified no warn was displayed.

--warn-section-align
    Warn if the address of an output section is changed because of
    alignment. Typically, the alignment will be set by an input section.
    The address will only be changed if it not explicitly specified;
    that is, if the SECTIONS command does not specify a start address
    for the section.

Fix stm32duino#465

Signed-off-by: Frederic.Pillon <[email protected]>
@fpistm fpistm added the fix 🩹 Bug fix label Mar 27, 2019
@fpistm fpistm added this to the 1.6.0 milestone Mar 27, 2019
@fpistm fpistm self-assigned this Mar 27, 2019
@fpistm fpistm merged commit 9299ab2 into stm32duino:master Mar 27, 2019
@fpistm fpistm deleted the ld_warning branch March 27, 2019 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 🩹 Bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant