Meta: Use release builds as the basis for Sanitizer builds
Some checks are pending
CI / Lagom (false, FUZZ, ubuntu-24.04, Linux, Clang) (push) Waiting to run
CI / Lagom (false, NO_FUZZ, macos-14, macOS, Clang) (push) Waiting to run
CI / Lagom (false, NO_FUZZ, ubuntu-24.04, Linux, GNU) (push) Waiting to run
CI / Lagom (true, NO_FUZZ, ubuntu-24.04, Linux, Clang) (push) Waiting to run
Package the js repl as a binary artifact / build-and-package (macos-14, macOS, macOS-universal2) (push) Waiting to run
Package the js repl as a binary artifact / build-and-package (ubuntu-24.04, Linux, Linux-x86_64) (push) Waiting to run
Run test262 and test-wasm / run_and_update_results (push) Waiting to run
Lint Code / lint (push) Waiting to run
Push notes / build (push) Waiting to run

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)
This commit is contained in:
Timothy Flynn 2024-10-26 10:27:05 -04:00 committed by Andreas Kling
parent ff3d78f369
commit 6cd224bb88
Notes: github-actions[bot] 2024-10-26 20:59:31 +00:00
2 changed files with 3 additions and 2 deletions

View file

@ -100,9 +100,9 @@
}, },
{ {
"name": "Sanitizer", "name": "Sanitizer",
"inherits": "Debug", "inherits": "default",
"displayName": "Sanitizer Config", "displayName": "Sanitizer Config",
"description": "Debug build using Sanitizers", "description": "Sanitizer build using Ninja generator",
"binaryDir": "${fileDir}/Build/ladybird-sanitizers", "binaryDir": "${fileDir}/Build/ladybird-sanitizers",
"cacheVariables": { "cacheVariables": {
"ENABLE_UNDEFINED_SANITIZER": "ON", "ENABLE_UNDEFINED_SANITIZER": "ON",

View file

@ -1,3 +1,4 @@
set(VCPKG_BUILD_TYPE release)
set(VCPKG_LIBRARY_LINKAGE static) set(VCPKG_LIBRARY_LINKAGE static)
set(VCPKG_C_FLAGS "") set(VCPKG_C_FLAGS "")