Skip to content

Commit 6cd224b

Browse files
trflynn89awesomekling
authored andcommitted
Meta: Use release builds as the basis for Sanitizer builds
LLVM recommends compiling with at least -O1 to have decent performance with sanitizers enabled. Indeed, this improves CI performance of LibWeb tests as follows: GCC on Linux: 160.61s to 119.68s (40.93s faster) Clang on Linux: 65.56s to 55.64s ( 9.92s faster)
1 parent ff3d78f commit 6cd224b

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

CMakePresets.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,9 +100,9 @@
100100
},
101101
{
102102
"name": "Sanitizer",
103-
"inherits": "Debug",
103+
"inherits": "default",
104104
"displayName": "Sanitizer Config",
105-
"description": "Debug build using Sanitizers",
105+
"description": "Sanitizer build using Ninja generator",
106106
"binaryDir": "${fileDir}/Build/ladybird-sanitizers",
107107
"cacheVariables": {
108108
"ENABLE_UNDEFINED_SANITIZER": "ON",

Meta/CMake/vcpkg/sanitizer-triplets/sanitizer.cmake

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
set(VCPKG_BUILD_TYPE release)
12
set(VCPKG_LIBRARY_LINKAGE static)
23

34
set(VCPKG_C_FLAGS "")

0 commit comments

Comments
 (0)