Skip to content

testrunner: do not load std.cfg in executor tests#8419

Merged
chrchr-github merged 1 commit intodanmar:mainfrom
firewave:test-exec-std
Apr 7, 2026
Merged

testrunner: do not load std.cfg in executor tests#8419
chrchr-github merged 1 commit intodanmar:mainfrom
firewave:test-exec-std

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

@firewave firewave commented Apr 7, 2026

No description provided.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 7, 2026

@firewave
Copy link
Copy Markdown
Collaborator Author

firewave commented Apr 7, 2026

A better solution would be to make it a shared pointer but that would be way more intrusive.

This greatly speeds up the FileSettings executor tests since that analysis needs to copy the settings which includes the library.

TestThreadExecutorFS::many_threads
Clang 22 689,859,031 -> 76,287,375

valgrund runtime

before
5,37s user 0,14s system 99% cpu 5,546 total

after
1,69s user 0,11s system 99% cpu 1,822 total

@chrchr-github chrchr-github merged commit 83fa5ad into danmar:main Apr 7, 2026
91 of 92 checks passed
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.

2 participants