-
Couldn't load subscription status.
- Fork 5.2k
Update build to use C++17 compiler standard level #118858
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.
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
|
A few TODOs:
|
|
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) |
Fixes #112419
Let's see if we break anything