[BEETLE] Add support for IAR compiler #2427
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The new code has been tested against the mbed test suite. The three tests that are going on timeout are passing on other compilers (i.e. GCC_ARM): |
|
/morph test |
|
@fvincenzo Can you add the following key to the |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 633 Test failed! |
|
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 637 Test failed! |
|
The test failure was unrelated, but the build looks good! @fvincenzo any idea about the RTOS test failures you posted above? I believe in the past those tests tend to fail when the device doesn't have enough heap space. If you rebase/merge your PR with the latest master, there should now be some code that checks and prints an error message if you run out of space (introduced with this PR: #2402). You'll need to run |
| *.o binary | ||
| *.lib binary | ||
| *.a binary | ||
| *.ar binary |
There was a problem hiding this comment.
How come this file is here? Can this be removed?
There was a problem hiding this comment.
I am adding binary libraries for the Cordio stack and platforms and I need git to know that are binaries. Any specific reason why I should remove it?
|
I am running on the latest version of the master. The PR is contained in my code. I am going to provide to you the test results via email. |
IAR compiler support is a mandatory requirement for mbed-os 5.1. This patch adds support to IAR on Beetle mbed-os platform. It contains: * Linker script * Startup code * Target enablement * Cordio libraries for BLE Signed-off-by: Douglas Raillard <douglas.raillard@arm.com> Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com>
mbedci-test CI server is complian only with mbed-os release version 5.x. This patch updates the Beetle section of the target.json file in order to comply with the requirement. Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com>
|
With the latest changes all the tests are passing. @0xc0170 @bridadan Could you please merge the PR? |
|
Great work @fvincenzo! /morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 638 Test failed! |
|
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 639 Test failed! |
|
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 640 Test failed! |
|
Tests are OK. The failure is for KL46Z, not relevant to this changeset. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
IAR compiler support is a mandatory requirement for mbed-os 5.1.
This patch adds support to IAR on Beetle mbed-os platform.
It contains:
Signed-off-by: Douglas Raillard douglas.raillard@arm.com
Signed-off-by: Vincenzo Frascino vincenzo.frascino@arm.com