Skip to content

Commit

Permalink
Merge pull request #73777 from myaaaaaaaaa/enable-tsan
Browse files Browse the repository at this point in the history
Add a Linux ThreadSanitizer job to CI
  • Loading branch information
YuriSizov committed Aug 4, 2023
2 parents c244903 + 8b78ad5 commit 16a9356
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 0 deletions.
11 changes: 11 additions & 0 deletions .github/workflows/linux_builds.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ env:
SCONSFLAGS: verbose=yes warnings=extra werror=yes module_text_server_fb_enabled=yes
DOTNET_NOLOGO: true
DOTNET_CLI_TELEMETRY_OPTOUT: true
TSAN_OPTIONS: suppressions=misc/error_suppressions/tsan.txt

concurrency:
group: ci-${{github.actor}}-${{github.head_ref || github.run_number}}-${{github.ref}}-linux
Expand Down Expand Up @@ -58,6 +59,16 @@ jobs:
# Skip 2GiB artifact speeding up action.
artifact: false

- name: Editor with ThreadSanitizer (target=editor, tests=yes, dev_build=yes, use_tsan=yes, use_llvm=yes, linker=lld)
cache-name: linux-editor-thread-sanitizer
target: editor
tests: true
sconsflags: dev_build=yes use_tsan=yes use_llvm=yes linker=lld
bin: "./bin/godot.linuxbsd.editor.dev.x86_64.llvm.san"
build-mono: false
# Skip 2GiB artifact speeding up action.
artifact: false

- name: Template w/ Mono (target=template_release)
cache-name: linux-template-mono
target: template_release
Expand Down
7 changes: 7 additions & 0 deletions misc/error_suppressions/tsan.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
# See the below link for an explanation of this file's format
# https://github.com/google/sanitizers/wiki/ThreadSanitizerSuppressions

deadlock:tests/core/templates/test_command_queue.h
deadlock:modules/text_server_adv/text_server_adv.cpp
deadlock:modules/text_server_fb/text_server_fb.cpp

0 comments on commit 16a9356

Please sign in to comment.