Skip to content

Commit c806ba8

Browse files
committed
fix(cpp-matrix): c++17 is supported by clang >=6
1 parent fa6ba7c commit c806ba8

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

cpp-matrix/dist/index.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

cpp-matrix/index.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -472,7 +472,8 @@ function compilerSupportsStd(compiler, version, cxxstd) {
472472
if (compiler === 'clang') {
473473
return (cxxstd <= 2023 && semver.satisfies(version, '>=17')) ||
474474
(cxxstd <= 2020 && semver.satisfies(version, '>=10')) ||
475-
(cxxstd <= 2017 && semver.satisfies(version, '>=5')) ||
475+
// clang >=5 technically supports c++17, but compliance is terrible
476+
(cxxstd <= 2017 && semver.satisfies(version, '>=6')) ||
476477
(cxxstd <= 2014 && semver.satisfies(version, '>=3.5')) ||
477478
(cxxstd <= 2011 && semver.satisfies(version, '>=3')) ||
478479
cxxstd <= 2003

0 commit comments

Comments
 (0)