Skip to content

[roottest] Temporarily disable test - #23345

Merged
vepadulano merged 1 commit into
root-project:masterfrom
vepadulano:disable-cpp11Containers
Sep 12, 2026
Merged

[roottest] Temporarily disable test#23345
vepadulano merged 1 commit into
root-project:masterfrom
vepadulano:disable-cpp11Containers

Conversation

@vepadulano

@vepadulano vepadulano commented Sep 11, 2026

Copy link
Copy Markdown
Member

The test is disabled as it sporadically fails due to a performance degradation in LLVM22. That could be fixed by backporting patches from LLVM23, e.g. as done by #23284.

The test coverage for the "unorderedMap" test is kept via "unorderedMapUnitTests", a fully-compiled googletest-based rewrite which had already been added to the test suite. The "Names" test is currently not being replaced.

The test is disabled as it sporadically fails due to a performance degradation in LLVM22. That could be fixed by backporting patches from LLVM23, e.g. as done by root-project#23284.

The test coverage for the "unorderedMap" test is kept via "unorderedMapTests", a fully-compiled googletest-based rewrite which had already been added to the test suite. The "Names" test is currently not being replaced.
@github-actions

Copy link
Copy Markdown

Test Results

    23 files      23 suites   3d 18h 17m 15s ⏱️
 3 879 tests  3 879 ✅ 0 💤 0 ❌
78 964 runs  78 964 ✅ 0 💤 0 ❌

Results for commit 7f98850.

@vepadulano
vepadulano merged commit 34aa530 into root-project:master Sep 12, 2026
31 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