Skip to content

arch/arm: replace the "-Wall" build option with the "-ghstd=last" #17856

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
Merged
xiaoxiang781216 merged 1 commit into from
Jan 14, 2026
Merged

arch/arm: replace the "-Wall" build option with the "-ghstd=last" #17856

xiaoxiang781216 merged 1 commit into from
Jan 14, 2026

Conversation

@extinguish
Copy link
Contributor

@extinguish extinguish commented Jan 13, 2026

Summary

Update Green Hills (GHS) toolchain warning configuration by replacing the deprecated -Wall flag with the recommended -ghstd=last option in both CMake and Makefile build systems. This change applies only when building with the GHS toolchain, while other toolchains continue to use -Wall.

Impact

  1. Aligns NuttX build configuration with Green Hills compiler best practices and documentation recommendations
  2. No impact on other toolchains (GCC, Clang, etc.) which continue to use -Wall

Testing

  1. Verified successful compilation with Green Hills toolchain using the new -ghstd=last flag
  2. Confirmed that the change is properly conditionalized and does not affect non-GHS toolchain builds

@extinguish
ghs: replace the "-Wall" build option with the "-ghstd=last"
81486c6 
According to the Greenhills documentation, the "-Wall" option is
deprecated for use. Instead, the "-ghstd=last" option is recommended.
Compared with "-Wall", the "-ghstd=last" option can capture more
warnings and has stricter rules.

Signed-off-by: guoshichao <[email protected]>
@github-actions github-actions bot added Arch: arm Issues related to ARM (32-bit) architecture Size: S The size of the change in this PR is small labels Jan 13, 2026
@jerpelea jerpelea changed the title ghs: replace the "-Wall" build option with the "-ghstd=last" arch/arm: replace the "-Wall" build option with the "-ghstd=last" Jan 14, 2026
@xiaoxiang781216 xiaoxiang781216 merged commit 49a57dd into apache:master Jan 14, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jerpelea jerpelea jerpelea approved these changes

@xiaoxiang781216 xiaoxiang781216 xiaoxiang781216 approved these changes

@anchao anchao Awaiting requested review from anchao anchao is a code owner

Assignees

No one assigned

Labels

Arch: arm Issues related to ARM (32-bit) architecture Size: S The size of the change in this PR is small

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@extinguish @jerpelea @xiaoxiang781216