Skip to content

Conversation

sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Oct 2, 2025

This failure was causing the emscripten-reeleases roller to fail.

The code didn't take into about that system_libs could also contains flags such as --whole-archive.

In this particular test the value of system_libs is:

['-lGL-getprocaddr', '-lal', '-lhtml5', '-lstubs-debug', '-lc-debug', '-ldlmalloc-debug', '-lcompiler_rt', '--whole-archive', '-lc++-debug-noexcept', '--no-whole-archive', '-lc++abi-debug-noexcept', '-lsockets', '--whole-archive', '-llsan_rt', '--no-whole-archive', '-llsan_common_rt', '-lsanitizer_common_rt']

Fixes: #25472

@sbc100 sbc100 requested review from juj and dschuff October 2, 2025 15:48
The code didn't take into about that system_libs could also contains
flags such as `--whole-archive`.

In this particular test the value of `system_libs` is:

```
['-lGL-getprocaddr', '-lal', '-lhtml5', '-lstubs-debug', '-lc-debug', '-ldlmalloc-debug', '-lcompiler_rt', '--whole-archive', '-lc++-debug-noexcept', '--no-whole-archive', '-lc++abi-debug-noexcept', '-lsockets', '--whole-archive', '-llsan_rt', '--no-whole-archive', '-llsan_common_rt', '-lsanitizer_common_rt']
```
@sbc100 sbc100 merged commit 507d31f into emscripten-core:main Oct 2, 2025
32 of 33 checks passed
@sbc100 sbc100 deleted the fix_lsan_test branch October 2, 2025 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

lsan.test_dylink_iostream failure
3 participants