Update build to use C++17 compiler standard level#118858
Merged
jkoritzinsky merged 5 commits intomainfrom Aug 19, 2025
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR updates the native build configuration to use C++17 as the compiler standard level instead of C++11. This change enables the use of modern C++17 language features and standard library components throughout the native codebase.
Key changes:
- Updates CMAKE_CXX_STANDARD from 11 to 17 in the native build configuration
This was referenced Aug 19, 2025
Member
|
A few TODOs:
|
Member
Author
|
I'll make the rapidjson change in this PR, but I want to hold off on more in-depth changes (like the riscv one) until after the upgrade PR is in (to keep this one small) |
max-charlamb
approved these changes
Aug 19, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #112419
Let's see if we break anything