-
Notifications
You must be signed in to change notification settings - Fork 407
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
Unify algorithm unit tests to avoid code duplication #3851
Conversation
d61291a
to
cb5f476
Compare
TestOpenMP.cpp | ||
LIST(APPEND ALGORITHM_SOURCES | ||
TestOpenMP_Sort1D.cpp | ||
TestOpenMP_Sort3D.cpp | ||
TestOpenMP_SortDynamicView.cpp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TestOpenMP.cpp
is merged into TestOpenMP_Sort1D.cpp
and TestOpenMP_Randonm.cpp
is the test corresponding to TestBACKEND.cpp
for the other backends.
@@ -42,36 +42,14 @@ | |||
//@HEADER | |||
*/ | |||
|
|||
#include <Kokkos_Macros.hpp> | |||
#ifdef KOKKOS_ENABLE_OPENMP | |||
#ifndef KOKKOS_TEST_CUDA_HPP |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These files are verbatim copied from core/unit_test
.
We could save a lot of LOC if we are to combine the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see the scope of this PR creeped quire a bit. I am sure we can improve even further but let's not go too crazy.
I would also be happy to split the pull request and combine all the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Personally I wouldn't worry about splitting it up, this looks like a good bit of work
Retest this please. |
This follows the same approach we are taking for
core/unit_test
.