-
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
Remove all deprecations in Kokkos 3 #5297
Remove all deprecations in Kokkos 3 #5297
Conversation
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.
Do the public header changes in a separate PR or at the very least in a separate commit
bb4d83f
to
d05ab09
Compare
core/src/Kokkos_Core_fwd.hpp
Outdated
#ifdef KOKKOS_ENABLE_DEPRECATED_CODE_3 | ||
struct InitArguments; | ||
#endif |
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.
Let's buy people more time on that one
@@ -105,16 +105,6 @@ using promote_2_t = typename promote_2<T, U>::type; | |||
#endif | |||
#endif | |||
|
|||
#if defined(KOKKOS_ENABLE_DEPRECATED_CODE_3) |
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.
For the math functions too we should allow for more time to transition
c5c3f32
to
b1d635d
Compare
b1d635d
to
ef943ab
Compare
ef943ab
to
5290902
Compare
Relevant changes in this pull request:
Consider reintroducing DEPRECATED_CODE_3 and use it for the code not removed above but moved to DEPRECATED_CODE_4 |
8a501b4
to
e645320
Compare
@@ -1,14 +0,0 @@ | |||
#include <pthread.h> | |||
|
|||
void* kokkos_test(void* args) { return args; } |
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.
Oversight forgot to remove that file in 9147621
partiton_master
deprecated.Kokkos_ENABLE_DISABLED_CODE_4
.