Skip to content

Commit

Permalink
Enable C++20 on Linux.
Browse files Browse the repository at this point in the history
Doing this in advance of enabling for more platforms allows monitoring
compile-time effects with less chance of adversely affecting the CQ,
while still getting the benefit of breaking a bot if someone lands a
change that breaks the build only in C++20.

Bug: 1284275
Change-Id: I88d6db79fad8d4552a2ee68b5d5b54914ca61042
Cq-Include-Trybots: chromium/try:linux_chromium_cfi_rel_ng
Cq-Include-Trybots: chromium/try:linux_chromium_msan_rel_ng
Cq-Include-Trybots: chromium/try:linux-official
Cq-Include-Trybots: chromium/try:linux-swangle-try-x64
Cq-Include-Trybots: chrome/try:linux-chrome
Cq-Include-Trybots: chrome/try:linux-pgo
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3880023
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1052163}
  • Loading branch information
pkasting authored and Chromium LUCI CQ committed Sep 28, 2022
1 parent 5489885 commit 4026b23
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions build/config/compiler/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -592,6 +592,9 @@ config("compiler") {
if (is_clang) {
cflags_cc += [ "-fno-trigraphs" ]
}
} else if (is_linux) {
# TODO(crbug.com/1284275): Switch to C++20 on all platforms.
cflags_cc += [ "-std=${standard_prefix}++20" ]
} else {
cflags_cc += [ "-std=${standard_prefix}++17" ]
}
Expand Down

0 comments on commit 4026b23

Please sign in to comment.