Skip to content

Makefile.cflags: Add warning for LTO#3376

Merged
jnohlgard merged 1 commit intoRIOT-OS:masterfrom
jnohlgard:pr/lto-warning
Jul 13, 2015
Merged

Makefile.cflags: Add warning for LTO#3376
jnohlgard merged 1 commit intoRIOT-OS:masterfrom
jnohlgard:pr/lto-warning

Conversation

@jnohlgard
Copy link
Copy Markdown
Member

LTO is currently broken on multiple platforms. This adds a warning until #3361 is merged.

@jnohlgard jnohlgard added Area: build system Area: Build system Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer labels Jul 13, 2015
@jnohlgard jnohlgard added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jul 13, 2015
@kaspar030
Copy link
Copy Markdown
Contributor

ACK. let's wait for travis, though.

@kaspar030
Copy link
Copy Markdown
Contributor

@phiros Why is strider not picking this up?

jnohlgard pushed a commit that referenced this pull request Jul 13, 2015
Makefile.cflags: Add warning for LTO
@jnohlgard jnohlgard merged commit bbe6547 into RIOT-OS:master Jul 13, 2015
@jnohlgard jnohlgard deleted the pr/lto-warning branch July 13, 2015 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: build system Area: Build system CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants