Skip to content

Commit

Permalink
Enable std_thread_scheduler to be tested with valgrind in CI
Browse files Browse the repository at this point in the history
  • Loading branch information
msimberg committed Oct 4, 2023
1 parent e7961b7 commit 332e62a
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions .github/workflows/linux_valgrind.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,7 @@ jobs:
shell: bash
run: |
cmake --build build --target tests.unit.modules.execution
cmake --build build --target std_thread_scheduler_test
- name: Test
shell: bash
run: |
Expand Down
6 changes: 3 additions & 3 deletions libs/pika/executors/tests/unit/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,8 @@ set(tests
thread_pool_scheduler
)

set(std_thread_scheduler_PARAMETERS VALGRIND)

# nvc++ causes test failures in release mode. This is assumed to be a code
# generation bug.
if(NOT CMAKE_CXX_COMPILER_ID STREQUAL "NVHPC")
Expand All @@ -33,8 +35,6 @@ endif()
foreach(test ${tests})
set(sources ${test}.cpp)

set(${test}_PARAMETERS THREADS 4)

source_group("Source Files" FILES ${sources})

set(folder_name "Tests/Unit/Modules/Executors")
Expand All @@ -47,6 +47,6 @@ foreach(test ${tests})
FOLDER ${folder_name}
)

pika_add_unit_test("modules.executors" ${test} ${${test}_PARAMETERS})
pika_add_unit_test("modules.executors" ${test} ${${test}_PARAMETERS} THREADS 4)

endforeach()

0 comments on commit 332e62a

Please sign in to comment.