Skip to content

cpu - sam3x8e: added switch for newlib-nano specs#1356

Merged
OlegHahm merged 1 commit intoRIOT-OS:masterfrom
haukepetersen:fix_sam3x_newlibnano
Jul 2, 2014
Merged

cpu - sam3x8e: added switch for newlib-nano specs#1356
OlegHahm merged 1 commit intoRIOT-OS:masterfrom
haukepetersen:fix_sam3x_newlibnano

Conversation

@haukepetersen
Copy link
Copy Markdown
Contributor

enable the newlib nano specs if they are available.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about this change?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The removed line is not needed - it was a leftover from some early testing... The added line contains some options, that are recommended when using newlib.nano, and I have found that they do not hurt for the default newlib...

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems you were wrong - with -flto building falls on my non-nano newlib version.

@OlegHahm
Copy link
Copy Markdown
Member

OlegHahm commented Jul 2, 2014

ACK and go.

OlegHahm added a commit that referenced this pull request Jul 2, 2014
cpu - sam3x8e: added switch for newlib-nano specs
@OlegHahm OlegHahm merged commit d52df83 into RIOT-OS:master Jul 2, 2014
@haukepetersen haukepetersen deleted the fix_sam3x_newlibnano branch July 24, 2014 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants