Skip to content

Commit 3716a76

Browse files
committed
[libc++] Run the test suite with optimizations by default
In production, libc++ is used with optimizations enabled 99.9% of the time. Therefore, I think it makes sense to run our tests with optimizations enabled by default, to be closer to what people actually use in production. It's still possible to disable optimizations for debugging purposes or other reasons. This also removes the pitfall of trying to run the benchmarks but forgetting to set the optimization level.
1 parent 79dab3f commit 3716a76

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

libcxx/utils/libcxx/test/params.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,7 @@ def getSuitableClangTidy(cfg):
193193
choices=["none", "speed", "size"],
194194
type=str,
195195
help="The optimization level to use when compiling the test suite.",
196-
default="none",
196+
default="speed",
197197
actions=lambda opt: filter(None, [
198198
AddCompileFlag(lambda cfg: getSpeedOptimizationFlag(cfg)) if opt == "speed" else None,
199199
AddCompileFlag(lambda cfg: getSizeOptimizationFlag(cfg)) if opt == "size" else None,

0 commit comments

Comments
 (0)