Skip to content

Commit 4e82614

Browse files
authored
Update C++ standard default to 17 (#376)
In accordance with Google's [Foundational C++ Support matrix](https://github.com/google/oss-policies-info/blob/main/foundational-cxx-support-matrix.md), and given other Google projects have moved to adopt C++17 as a minimum or default ([protobuf](protocolbuffers/protobuf@fe53593), [rules_cc](https://github.com/search?q=repo%3Abazelbuild%2Frules_cc+%22-std%22&type=code)), I'd like us to consider bumping this version in apple_support as well. This constitutes a breaking change. It uses the change made in #219 as a reference.
1 parent 92cef2e commit 4e82614

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

crosstool/cc_toolchain_config.bzl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -447,7 +447,7 @@ please file an issue at https://github.com/bazelbuild/apple_support/issues/new
447447
"-target",
448448
target_system_name,
449449
"-stdlib=libc++",
450-
"-std=gnu++14",
450+
"-std=gnu++17",
451451
],
452452
),
453453
],
@@ -2084,7 +2084,7 @@ please file an issue at https://github.com/bazelbuild/apple_support/issues/new
20842084
ACTION_NAMES.lto_backend,
20852085
ACTION_NAMES.clif_match,
20862086
],
2087-
flag_groups = [flag_group(flags = ["-std=c++14"])],
2087+
flag_groups = [flag_group(flags = ["-std=c++17"])],
20882088
),
20892089
],
20902090
)

0 commit comments

Comments
 (0)