Skip to content

Upgrade C++ standard from C++17 to C++20 - #348

Open
laurettaSchubert wants to merge 2 commits into
Xilinx:main-gefrom
laurettaSchubert:lauretta/upgrade-cxx-standard-to-cpp20
Open

Upgrade C++ standard from C++17 to C++20#348
laurettaSchubert wants to merge 2 commits into
Xilinx:main-gefrom
laurettaSchubert:lauretta/upgrade-cxx-standard-to-cpp20

Conversation

@laurettaSchubert

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

1 similar comment
@github-actions

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@laurettaSchubert
laurettaSchubert force-pushed the lauretta/upgrade-cxx-standard-to-cpp20 branch from db2c741 to ce3a618 Compare August 12, 2026 12:31
Align with XMC toolchain upgrade (XMC PR #4602).

Signed-off-by: Lauretta Klein <lauretta.klein@amd.com>
GCC 15 (ubuntu-26.04) emits a spurious warning when aggressive inlining
of vector::push_back confuses its alias analysis. Suppress with a
targeted pragma around the affected function only.

Signed-off-by: Lauretta Klein <lauretta.klein@amd.com>
@laurettaSchubert
laurettaSchubert force-pushed the lauretta/upgrade-cxx-standard-to-cpp20 branch from ce3a618 to 2d55aeb Compare August 12, 2026 12:39
@github-actions

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@stsoe stsoe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are still supporting compilers that are not c++20 compatible, so you can't use some newer C++ features, e.g. std::span, std::format, etc. It'd be best to stick with C++17 until legacy compilers are done with.

@sonals

sonals commented Aug 14, 2026

Copy link
Copy Markdown
Member

aiebu is also used on somewhat dated systems with older compilers for several products. Moving to C++20 as baseline is not feasible at this time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants