-
Notifications
You must be signed in to change notification settings - Fork 178
Add ARMC6 where appropriate #205
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One tiny typo
Otherwise, this looks good. 👍
@@ -72,7 +72,7 @@ There are other entries that require a bit of clarification: | |||
|
|||
#### Current support | |||
|
|||
We have tested *memap* on Windows 7, Linux and Mac OS X. The `GCC_ARM` (GNU Arm Embedded Toolchain), Arm (Arm Compiler 5) and IAR toolchains generate memory map files. | |||
We have tested *memap* on Windows 7, Linux and Mac OS X. The `GCC_ARM` (GNU Arm Embedded Toolchain), Arm Compiler 5, Arm Compiler 6 and IAR toolchains generate memory map files. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please delete the extra space after "and".
34dc238
to
d587c54
Compare
@theotherjimmy Please resolve conflicts if this is ready for me to merge. |
d587c54
to
d057c68
Compare
Rebase resolved it trivially, apparently. |
Just a sec, removing out of date info |
d057c68
to
861bf5d
Compare
Done. |
|
Sorry for the noise. It's all correct now. |
DO NOT MERGE BEFORE ARMmbed/mbed-os#4949 IS MERGED