Fix multiple definitions of '__aeabi_atexit' #9126

fluidblue · 2018-12-17T14:07:27Z

Discussed in #9123
Fix recommended by @0xc0170

[X] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

0xc0170 · 2018-12-17T14:11:22Z

I've noticed GCC_CR commit, that one can be removed and even that startup file (entire folder). That toolchain was removed

cmonr · 2018-12-17T16:57:11Z

Waiting on #9131 for a fix to Travis CI.

cmonr · 2018-12-17T16:57:43Z

@fluidblue Thanks for keeping the changes along commit boundaries!

mbed-ci · 2018-12-18T10:21:40Z

Test run: FAILED

Summary: 1 of 11 test jobs failed
Build number : 1
Build artifacts

Failed test jobs:

0xc0170 · 2018-12-18T10:38:50Z

Test restarted (issue is not related and still needs to be reviewed on master)

fluidblue added 2 commits December 17, 2018 15:00

Prevent multiple __aeabi_atexit definition linker error (GCC_ARM)

69ef804

Prevent multiple __aeabi_atexit definition linker error (GCC_CR)

c9eb60c

0xc0170 approved these changes Dec 17, 2018

View reviewed changes

Removed GCC_CR toolchain for LPC11U68

96b3454

0xc0170 approved these changes Dec 17, 2018

View reviewed changes

cmonr added the needs: preceding PR label Dec 17, 2018

cmonr approved these changes Dec 17, 2018

View reviewed changes

0xc0170 added needs: CI and removed needs: preceding PR labels Dec 18, 2018

0xc0170 mentioned this pull request Dec 18, 2018

K82F QSPI test fails intermittently #9075

Closed

cmonr added ready for merge release-version: 5.11.1 and removed needs: CI labels Dec 18, 2018

cmonr merged commit 15765d6 into ARMmbed:master Dec 18, 2018

cmonr removed the ready for merge label Dec 18, 2018