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.

Sign up for GitHub

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

Jump to bottom

[csolution] cdefault: GCC inline assembly shall accept unified syntax by default #1057

Closed
JonatanAntoni opened this issue Jul 11, 2023 · 1 comment · Fixed by #1045
Closed

[csolution] cdefault: GCC inline assembly shall accept unified syntax by default #1057

JonatanAntoni opened this issue Jul 11, 2023 · 1 comment · Fixed by #1045
Assignees
@JonatanAntoni
Labels
enhancement New feature or request

Comments

@JonatanAntoni
Copy link
Member

JonatanAntoni commented Jul 11, 2023 •
edited
Loading

Describe The Problem To Be Solved

For inline assembly we typically use unified syntax to be as compiler-agnostic as possible. GCC still uses disjoint (i.e., pre-unified) syntax by default for history reasons. This could make compilation units using unified syntax fail to compile.

Suggest A Solution

GCC has the command line flag -masm-syntax-unified to switch to unified syntax. This flag shall be used by default, e.g., by adding it into cdefault.yml.
@JonatanAntoni JonatanAntoni added the enhancement New feature or request label Jul 11, 2023
@JonatanAntoni
Copy link
Member Author

Fixed in #1045.

@jkrech jkrech changed the title GCC inline assembly shall accept unified syntax by default [csolution] cdefault: GCC inline assembly shall accept unified syntax by default Jul 11, 2023
@jkrech jkrech linked a pull request Jul 18, 2023 that will close this issue
projmgr: Add -masm-syntax-unified to GCC cdefault #1045
Merged
@jkrech jkrech closed this as completed Jul 18, 2023
grasci-arm pushed a commit to ARM-software/devtools that referenced this issue Oct 31, 2024
@soumeh01
Workflows security updates (Open-CMSIS-Pack#1057)
4da4b29
jkrech pushed a commit that referenced this issue Oct 31, 2024
@spcaipers-arm @soumeh01
Workflows security updates (#1057) (#1835)
5c2025f 
Co-authored-by: Sourabh Mehta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JonatanAntoni JonatanAntoni

Labels
enhancement New feature or request
Projects
No open projects
CMSIS-Toolbox 2.1
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

projmgr: Add -masm-syntax-unified to GCC cdefault ARM-software/devtools
2 participants
@jkrech @JonatanAntoni