Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature armc6 branch rebase - test #9810

Closed
wants to merge 10 commits into from

Conversation

0xc0170
Copy link
Contributor

@0xc0170 0xc0170 commented Feb 22, 2019

Description

Testing ARMC6 feature branch rebase

cc @SenRamakri (we are rebasing to get latest networking in, wiced to be rebuilt).

Pull request type

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

Reviewers

Release Notes

SenRamakri and others added 10 commits February 22, 2019 08:56
Assembler in rtx_core_ca.h is on the whole unified syntax such that it
should work fine if assembled as either ARM or Thumb-2.

The exception was __get_PSP, which uses STM^, which is only available
in ARM state.

Flagging for this function was incorrect, except for IAR - it switched
assembler state without telling the compiler, meaning that the resulting
assembler output could be incorrect, and that the function itself would
not be correctly marked as an ARM function - the CPU would enter in
Thumb state.

Alternative fix would be to switch to System mode, which would work
as either ARM or Thumb-2 assembler, like the rest of the file, but
this is the minimal change.

Fixes ARMmbed#526.
@0xc0170 0xc0170 changed the base branch from master to feature-armc6 February 22, 2019 09:02
@0xc0170
Copy link
Contributor Author

0xc0170 commented Feb 22, 2019

Ci started

@ciarmcom ciarmcom requested a review from a team February 22, 2019 10:00
@ciarmcom
Copy link
Member

@0xc0170, thank you for your changes.
@ARMmbed/mbed-os-maintainers please review.

@mbed-ci
Copy link

mbed-ci commented Feb 22, 2019

Test run: SUCCESS

Summary: 12 of 12 test jobs passed
Build number : 1
Build artifacts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants