Documentation updates for Toolchain changes #945
Conversation
SenRamakri
requested review from
AnotherButler,
deepikabhavnani and
kegilbert
|
Please review - But DO NOT MERGE this until the compiler changes are done. We will update the comments in the PR once we are ready to merge this after tool chain work is done. I took a shortcut as of now to capture both IAR and ARMC changes in one PR. Its possible that one might happen but other might not, in that case we will update this PR accordingly. This covers - IOTCORE-825 and IOTCORE-709. |
|
Iar doc update PR was created long back - #885 In case one is updated can use that |
SenRamakri
force-pushed
the
sen_CompilerUpdateDocs
branch
from
f22e192 to
5416a23
Compare
|
@AnotherButler @kegilbert - I added more changes to this PR as we changed our requirements around ARMC6 transition and that caused more documentation changes. Please review. |
|
Does this still have a code dependency? |
|
@SenRamakri Is this ready to merge, or are we waiting on a code dependency? |
@AnotherButler - Unfortunately we need to wait, as IAR8 just merged today and we have to start on ARMC6 tomorrow. Sorry that we have to keep you waiting on this. |
|
Could you please add a link here to the code PR? |
|
@AnotherButler - Yes, actually we don't have a specific PR, we need the branch feature-armc6 merged before we can merge this documentation change. In fact, the merge PR is still not generated and Im waiting on this PR now - ARMmbed/mbed-os#9860, sorry for the convoluted info. |
| The `supported_toolchains` property is the list of toolchains that support a target. The allowed values for `supported_toolchains` are `ARM`, `uARM`, `ARMC6`, `GCC_ARM` and `IAR`. | ||
| The `supported_toolchains` property is the list of toolchains that support a target. The allowed values for `supported_toolchains` are `ARM`, `uARM`, `ARMC5`, `ARMC6`, `GCC_ARM` and `IAR`. | ||
|
|
||
| If you specify `ARMC5` in `supported_toolchains`, it means the corresponding target supports `Arm Compiler 5.06 update 6`. |
There was a problem hiding this comment.
We should be explicit that any new targets should support ARMC6 not 5.
There was a problem hiding this comment.
@bulislaw - Please see the updates.
|
Needs a rebase to resolve conflicts |
Make minor edits to include articles for consistency.
Edit file for minor formatting.
Edit file for consistent capitalization and formatting.
SenRamakri
force-pushed
the
sen_CompilerUpdateDocs
branch
from
d23347e to
b0b086b
Compare
|
Rebased |
Edit note, mostly for branding and brevity.
Edit note, mostly for branding and active voice.
|
@bulislaw Have all of your concerns been addressed? |
|
Master received ARMC6 update. We are close to 5.12rc1, this documentation should be part of it. |
|
@AnotherButler - ARMC6/IAR8 changes are merged to master. I think we are good to merge these changes now. |
|
@bulislaw I'm going to merge this later today. If you have further concerns, please create a new PR. |
Documentation updates for Toolchain changes planned for 5.12 release