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

Add top-level makefile option to just test kokkos-core unit-test #485

Closed
crtrott opened this issue Oct 19, 2016 · 1 comment
Closed

Add top-level makefile option to just test kokkos-core unit-test #485

crtrott opened this issue Oct 19, 2016 · 1 comment
Assignees
@crtrott
Labels
Enhancement Improve existing capability; will potentially require voting
Milestone
Fall 2016

Comments

@crtrott
Copy link
Member

crtrott commented Oct 19, 2016

This is to maximize testing per time for compiler vendors. The Kokkos core unit-tests should catch everything in the core library and it is rare to have bugs not caught in there if they are caused by compiler issues.

We then would recommend to compilers to include that option in their compiler test.
@crtrott crtrott added the Enhancement Improve existing capability; will potentially require voting label Oct 19, 2016
@crtrott crtrott self-assigned this Oct 19, 2016
@crtrott crtrott added this to the Fall 2016 milestone Oct 19, 2016
@crtrott
Copy link
Member Author

crtrott commented Oct 23, 2016 •
edited

OK I got an option now which test unit-tests of core, containers and algorithms. Build + Test on my machine is 3:06 for Cuda with NVCC 8.0 (2:23 with clang) and 1:07 for OpenMP with GCC 5.3. Need to ask compiler vendors if this is ok for inclusion in their test suites, or whether we should restrict to core (which probably cuts it by another 2x).

crtrott added a commit that referenced this issue Oct 23, 2016
@crtrott
GenerateMakefile: add option to do unit-tests-only
9820560 
The purpose of this is to maximize testing per time for
compiler vendors who want to make Kokkos testing part of their
nightlies.

Addresses issue #485
@crtrott crtrott closed this as completed Oct 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@crtrott crtrott

Labels
Enhancement Improve existing capability; will potentially require voting
Projects
None yet
Milestone
Fall 2016
Development

No branches or pull requests
1 participant
@crtrott