Skip to content

bump ARM toolchain to 9-2019-q4#78

Merged
kaspar030 merged 1 commit intoRIOT-OS:masterfrom
kaspar030:bump_arm_toolchain_8_2019_q3
Feb 24, 2020
Merged

bump ARM toolchain to 9-2019-q4#78
kaspar030 merged 1 commit intoRIOT-OS:masterfrom
kaspar030:bump_arm_toolchain_8_2019_q3

Conversation

@kaspar030
Copy link
Copy Markdown
Contributor

@kaspar030 kaspar030 commented Aug 20, 2019

This mostly brings arm gcc to gcc 8.

The PR also somewhat parameterizes the URL so future updates will be a little easier.

@kaspar030 kaspar030 force-pushed the bump_arm_toolchain_8_2019_q3 branch from 1c1e1d2 to 723b23a Compare August 20, 2019 10:25
@kaspar030
Copy link
Copy Markdown
Contributor Author

This seems to trigger a warning in rmutex.c...

@benpicco
Copy link
Copy Markdown
Contributor

What's the warning?

@kaspar030 kaspar030 force-pushed the bump_arm_toolchain_8_2019_q3 branch from 723b23a to e2ba015 Compare February 10, 2020 12:36
@kaspar030 kaspar030 changed the title bump ARM toolchain to 7-2019-q3 bump ARM toolchain to 9-2019-q4 Feb 10, 2020
@kaspar030
Copy link
Copy Markdown
Contributor Author

What's the warning?

I don't remember. Seems to be gone with 9-2019-q4!

@kaspar030
Copy link
Copy Markdown
Contributor Author

  • rebased
  • updated to 9-2019-q4

Copy link
Copy Markdown
Contributor

@benpicco benpicco left a comment

Choose a reason for hiding this comment

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

Newer gcc will detect more errors, we should have this.

@kaspar030 kaspar030 force-pushed the bump_arm_toolchain_8_2019_q3 branch from e2ba015 to 595aad7 Compare February 24, 2020 08:48
@kaspar030
Copy link
Copy Markdown
Contributor Author

I'll do one more local test run with this

@kaspar030
Copy link
Copy Markdown
Contributor Author

I'll do one more local test run with this

the rmutex issue is definitely gone. unittests compile fine (tested nrf52840dk). let's go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants