Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[CMake] Mark multi-threaded tests as such #6810

Merged
merged 6 commits into from
Jun 27, 2022
Merged

[CMake] Mark multi-threaded tests as such #6810

merged 6 commits into from
Jun 27, 2022

Commits on Jun 20, 2022

  1. [NFC] Sort tests in tests() invocations alphabetically

    @LebedevRI
    LebedevRI committed Jun 20, 2022
    Configuration menu
    Copy the full SHA
    7b0328c View commit details
    Browse the repository at this point in the history

  2. [CMake] Add all tests that call halide_spawn_thread() into `multith… 

    …readed` group
    @LebedevRI
    LebedevRI committed Jun 20, 2022
    Configuration menu
    Copy the full SHA
    d3c3447 View commit details
    Browse the repository at this point in the history

  3. [CMake] Add tests that use std::thread into multithreaded group

    @LebedevRI
    LebedevRI committed Jun 20, 2022
    Configuration menu
    Copy the full SHA
    5510742 View commit details
    Browse the repository at this point in the history

  4. [CMake] Set HL_NUM_THREADS=1 for non-multithreaded tests 

    This actually prevents bit-rot for the new tests.
    @LebedevRI
    LebedevRI committed Jun 20, 2022
    Configuration menu
    Copy the full SHA
    27dfec3 View commit details
    Browse the repository at this point in the history

Commits on Jun 23, 2022

  1. [CMake] Separate labels with ; instead of

    @LebedevRI
    LebedevRI committed Jun 23, 2022
    Configuration menu
    Copy the full SHA
    1833a1d View commit details
    Browse the repository at this point in the history

Commits on Jun 24, 2022

  1. Revert "[CMake] Set HL_NUM_THREADS=1 for non-multithreaded tests" 

    I *do* believe that should be done, but clearly that is not the opinion
that the upstream shares.

This reverts commit 27dfec3.
    @LebedevRI
    LebedevRI committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    9d088f7 View commit details
    Browse the repository at this point in the history