Disable TSan for libstdc++ versions not handling it

TedLyngmo · TedLyngmo · commit 5af62b708f42 · 2026-01-06T09:44:55.000+01:00
Signed-off-by: Ted Lyngmo &lt;ted@lyncon.se&gt;
diff --git a/.github/workflows/ci_tests.yml b/.github/workflows/ci_tests.yml
@@ -42,29 +42,29 @@ jobs:
                   "tests": [
                     { "stdlibs": ["libstdc++"],
                       "tests": [
-                        "Debug.Default", "Release.Default", "Release.TSan",
+                        "Debug.Default", "Release.Default",
                         "Release.MaxSan", "Debug.Werror", "Debug.Dynamic",
                         "Debug.Coverage"
                       ]
                     }
                   ]
                 },
-                { "cxxversions": ["c++23", "c++20", "c++17"],
+                { "cxxversions": ["c++23", "c++20"],
                   "tests": [{ "stdlibs": ["libstdc++"], "tests": ["Release.Default"]}]
                 }
               ]
             },
             { "versions": ["14", "13"],
               "tests": [
-                { "cxxversions": ["c++26", "c++23", "c++20", "c++17"],
+                { "cxxversions": ["c++26", "c++23", "c++20"],
                   "tests": [{ "stdlibs": ["libstdc++"], "tests": ["Release.Default"]}]
                 }
               ]
             },
             {
               "versions": ["12", "11"],
               "tests": [
-                { "cxxversions": ["c++23", "c++20", "c++17"],
+                { "cxxversions": ["c++23", "c++20"],
                   "tests": [{ "stdlibs": ["libstdc++"], "tests": ["Release.Default"]}]
                 }
               ]
@@ -75,15 +75,21 @@ jobs:
               "tests": [
                 {"cxxversions": ["c++26"],
                   "tests": [
-                    { "stdlibs": ["libstdc++", "libc++"],
+                    { "stdlibs": ["libc++"],
                       "tests": [
                         "Debug.Default", "Release.Default", "Release.TSan",
                         "Release.MaxSan", "Debug.Werror", "Debug.Dynamic"
                       ]
+                    },
+                    { "stdlibs": ["libstdc++"],
+                      "tests": [
+                        "Debug.Default", "Release.Default",
+                        "Release.MaxSan", "Debug.Werror", "Debug.Dynamic"
+                      ]
                     }
                   ]
                 },
-                { "cxxversions": ["c++23", "c++20", "c++17"],
+                { "cxxversions": ["c++23", "c++20"],
                   "tests": [
                     {"stdlibs": ["libstdc++", "libc++"], "tests": ["Release.Default"]}
                   ]
@@ -92,7 +98,7 @@ jobs:
             },
             { "versions": ["20", "19"],
               "tests": [
-                { "cxxversions": ["c++26", "c++23", "c++20", "c++17"],
+                { "cxxversions": ["c++26", "c++23", "c++20"],
                   "tests": [
                     {"stdlibs": ["libstdc++", "libc++"], "tests": ["Release.Default"]}
                   ]
@@ -101,10 +107,10 @@ jobs:
             },
             { "versions": ["18", "17"],
               "tests": [
-                { "cxxversions": ["c++26", "c++23", "c++20", "c++17"],
+                { "cxxversions": ["c++26", "c++23", "c++20"],
                   "tests": [{"stdlibs": ["libc++"], "tests": ["Release.Default"]}]
                 },
-                { "cxxversions": ["c++20", "c++17"],
+                { "cxxversions": ["c++20"],
                   "tests": [{"stdlibs": ["libstdc++"], "tests": ["Release.Default"]}]
                 }
               ]
@@ -113,7 +119,7 @@ jobs:
           "appleclang": [
             { "versions": ["latest"],
               "tests": [
-                { "cxxversions": ["c++26", "c++23", "c++20", "c++17"],
+                { "cxxversions": ["c++26", "c++23", "c++20"],
                   "tests": [{ "stdlibs": ["libc++"], "tests": ["Release.Default"]}]
                 }
               ]

Original file line number	Diff line number	Diff line change
`@@ -42,29 +42,29 @@ jobs:`
`42`	`42`	`"tests": [`
`43`	`43`	`{ "stdlibs": ["libstdc++"],`
`44`	`44`	`"tests": [`
`45`		`- "Debug.Default", "Release.Default", "Release.TSan",`
	`45`	`+ "Debug.Default", "Release.Default",`
`46`	`46`	`"Release.MaxSan", "Debug.Werror", "Debug.Dynamic",`
`47`	`47`	`"Debug.Coverage"`
`48`	`48`	`]`
`49`	`49`	`}`
`50`	`50`	`]`
`51`	`51`	`},`
`52`		`- { "cxxversions": ["c++23", "c++20", "c++17"],`
	`52`	`+ { "cxxversions": ["c++23", "c++20"],`
`53`	`53`	`"tests": [{ "stdlibs": ["libstdc++"], "tests": ["Release.Default"]}]`
`54`	`54`	`}`
`55`	`55`	`]`
`56`	`56`	`},`
`57`	`57`	`{ "versions": ["14", "13"],`
`58`	`58`	`"tests": [`
`59`		`- { "cxxversions": ["c++26", "c++23", "c++20", "c++17"],`
	`59`	`+ { "cxxversions": ["c++26", "c++23", "c++20"],`
`60`	`60`	`"tests": [{ "stdlibs": ["libstdc++"], "tests": ["Release.Default"]}]`
`61`	`61`	`}`
`62`	`62`	`]`
`63`	`63`	`},`
`64`	`64`	`{`
`65`	`65`	`"versions": ["12", "11"],`
`66`	`66`	`"tests": [`
`67`		`- { "cxxversions": ["c++23", "c++20", "c++17"],`
	`67`	`+ { "cxxversions": ["c++23", "c++20"],`
`68`	`68`	`"tests": [{ "stdlibs": ["libstdc++"], "tests": ["Release.Default"]}]`
`69`	`69`	`}`
`70`	`70`	`]`
`@@ -75,15 +75,21 @@ jobs:`
`75`	`75`	`"tests": [`
`76`	`76`	`{"cxxversions": ["c++26"],`
`77`	`77`	`"tests": [`
`78`		`- { "stdlibs": ["libstdc++", "libc++"],`
	`78`	`+ { "stdlibs": ["libc++"],`
`79`	`79`	`"tests": [`
`80`	`80`	`"Debug.Default", "Release.Default", "Release.TSan",`
`81`	`81`	`"Release.MaxSan", "Debug.Werror", "Debug.Dynamic"`
`82`	`82`	`]`
	`83`	`+ },`
	`84`	`+ { "stdlibs": ["libstdc++"],`
	`85`	`+ "tests": [`
	`86`	`+ "Debug.Default", "Release.Default",`
	`87`	`+ "Release.MaxSan", "Debug.Werror", "Debug.Dynamic"`
	`88`	`+ ]`
`83`	`89`	`}`
`84`	`90`	`]`
`85`	`91`	`},`
`86`		`- { "cxxversions": ["c++23", "c++20", "c++17"],`
	`92`	`+ { "cxxversions": ["c++23", "c++20"],`
`87`	`93`	`"tests": [`
`88`	`94`	`{"stdlibs": ["libstdc++", "libc++"], "tests": ["Release.Default"]}`
`89`	`95`	`]`
`@@ -92,7 +98,7 @@ jobs:`
`92`	`98`	`},`
`93`	`99`	`{ "versions": ["20", "19"],`
`94`	`100`	`"tests": [`
`95`		`- { "cxxversions": ["c++26", "c++23", "c++20", "c++17"],`
	`101`	`+ { "cxxversions": ["c++26", "c++23", "c++20"],`
`96`	`102`	`"tests": [`
`97`	`103`	`{"stdlibs": ["libstdc++", "libc++"], "tests": ["Release.Default"]}`
`98`	`104`	`]`
`@@ -101,10 +107,10 @@ jobs:`
`101`	`107`	`},`
`102`	`108`	`{ "versions": ["18", "17"],`
`103`	`109`	`"tests": [`
`104`		`- { "cxxversions": ["c++26", "c++23", "c++20", "c++17"],`
	`110`	`+ { "cxxversions": ["c++26", "c++23", "c++20"],`
`105`	`111`	`"tests": [{"stdlibs": ["libc++"], "tests": ["Release.Default"]}]`
`106`	`112`	`},`
`107`		`- { "cxxversions": ["c++20", "c++17"],`
	`113`	`+ { "cxxversions": ["c++20"],`
`108`	`114`	`"tests": [{"stdlibs": ["libstdc++"], "tests": ["Release.Default"]}]`
`109`	`115`	`}`
`110`	`116`	`]`
`@@ -113,7 +119,7 @@ jobs:`
`113`	`119`	`"appleclang": [`
`114`	`120`	`{ "versions": ["latest"],`
`115`	`121`	`"tests": [`
`116`		`- { "cxxversions": ["c++26", "c++23", "c++20", "c++17"],`
	`122`	`+ { "cxxversions": ["c++26", "c++23", "c++20"],`
`117`	`123`	`"tests": [{ "stdlibs": ["libc++"], "tests": ["Release.Default"]}]`
`118`	`124`	`}`
`119`	`125`	`]`