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.

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

Limit CTEST_PARALLEL_LEVEL to 8 for GPU tests #5163

Merged
merged 1 commit into from
Mar 18, 2021

Conversation

GMNGeoffrey
Copy link
Contributor

@GMNGeoffrey GMNGeoffrey commented Mar 18, 2021

This should hopefully avoid exhausting GPU memory while testing.
Unfortunately, it will also slow down testing.
#5162 for more robust handling.

@google-cla google-cla bot added the cla: yes label Mar 18, 2021
@GMNGeoffrey GMNGeoffrey requested a review from ScottTodd March 18, 2021 19:25
@GMNGeoffrey GMNGeoffrey added infrastructure Relating to build systems, CI, or testing infrastructure/cmake labels Mar 18, 2021
@GMNGeoffrey GMNGeoffrey merged commit e3cdd5c into iree-org:main Mar 18, 2021
@GMNGeoffrey GMNGeoffrey deleted the ctest-parallelism branch March 18, 2021 19:35
@not-jenni not-jenni mentioned this pull request Mar 19, 2021
antiagainst added a commit to antiagainst/iree that referenced this pull request Apr 15, 2021
This reverts iree-org#5163 and
iree-org#5166 as now buffer usage
is improved on Vulkan.
antiagainst added a commit that referenced this pull request Apr 16, 2021
This reverts #5163 and
#5166 as now buffer usage
is improved on Vulkan.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Relating to build systems, CI, or testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants