From 7f988509069a2ac37f82d78de639d75893a44282 Mon Sep 17 00:00:00 2001 From: Vincenzo Eduardo Padulano Date: Fri, 11 Sep 2026 16:27:27 +0200 Subject: [PATCH] [roottest] Temporarily disable test 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 https://github.com/root-project/root/pull/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. --- .../root/io/cpp11Containers/CMakeLists.txt | 28 ++++++++++--------- 1 file changed, 15 insertions(+), 13 deletions(-) diff --git a/roottest/root/io/cpp11Containers/CMakeLists.txt b/roottest/root/io/cpp11Containers/CMakeLists.txt index d3bf41d3c0713..fa266b5090524 100644 --- a/roottest/root/io/cpp11Containers/CMakeLists.txt +++ b/roottest/root/io/cpp11Containers/CMakeLists.txt @@ -3,12 +3,13 @@ ROOTTEST_GENERATE_REFLEX_DICTIONARY(unorderedMap SELECTION unorderedMap_selection.xml FIXTURES_SETUP root-io-cpp11Containers-unorderedMapDict-fixture) -ROOTTEST_ADD_TEST(unorderedMap - MACRO execUnorderedMap.C - OUTREF execUnorderedMap.ref - FIXTURES_REQUIRED root-io-cpp11Containers-unorderedMapDict-fixture - FIXTURES_SETUP root-io-cpp11Containers-unorderedMap-fixture - LABELS longtest) +# Test temporarily disabled due to a performance regression in LLVM22 leading to sporadic failures. +# ROOTTEST_ADD_TEST(unorderedMap +# MACRO execUnorderedMap.C +# OUTREF execUnorderedMap.ref +# FIXTURES_REQUIRED root-io-cpp11Containers-unorderedMapDict-fixture +# FIXTURES_SETUP root-io-cpp11Containers-unorderedMap-fixture +# LABELS longtest) # Copy of the test above, written as a compiled C++ program based on gtest. Useful for more fine-grained view on # potential test failures @@ -47,13 +48,14 @@ if(NOT 32BIT AND NOT MSVC) root-io-cpp11Containers-forwardList-fixture FIXTURES_SETUP root-io-cpp11Containers-unorderedSet-fixture) - ROOTTEST_ADD_TEST(Names - COPY_TO_BUILDDIR auxCode.h commonUtils.h - MACRO execcpp11ContainersNames.C - OUTREF execcpp11ContainersNames.ref - FIXTURES_REQUIRED root-io-cpp11Containers-unorderedSet-fixture - root-io-cpp11Containers-forwardList-fixture - root-io-cpp11Containers-unorderedMap-fixture) +# Test temporarily disabled due to a performance regression in LLVM22 leading to sporadic failures. +# ROOTTEST_ADD_TEST(Names +# COPY_TO_BUILDDIR auxCode.h commonUtils.h +# MACRO execcpp11ContainersNames.C +# OUTREF execcpp11ContainersNames.ref +# FIXTURES_REQUIRED root-io-cpp11Containers-unorderedSet-fixture +# root-io-cpp11Containers-forwardList-fixture +# root-io-cpp11Containers-unorderedMap-fixture) endif()