-
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
Deprecate backward compatibility features #3978
Deprecate backward compatibility features #3978
Conversation
Please rebase |
8259955
to
4006af9
Compare
Rebased. |
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.
Make sure you list what is being deprecated in the description.
Consider handling MDRangePolicy elsewhere
example/tutorial/06_simple_mdrangepolicy/simple_mdrangepolicy.cpp
Outdated
Show resolved
Hide resolved
@@ -56,7 +56,7 @@ | |||
#include <Kokkos_OpenMPTargetSpace.hpp> | |||
#include <Kokkos_ScratchSpace.hpp> | |||
#include <Kokkos_Parallel.hpp> | |||
#include <Kokkos_TaskPolicy.hpp> | |||
#include <Kokkos_TaskScheduler.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.
Was this intentional?
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.
Yes, that file just forwards to Kokkos_TaskScheduler.hpp
:
kokkos/core/src/Kokkos_TaskPolicy.hpp
Lines 45 to 47 in 3efc176
// For backward compatibility: | |
#include <Kokkos_TaskScheduler.hpp> |
I didn't want to deal with issuing a deprecation warning when using this file but I also don't see a reason to use a deprecated file.
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.
Yeah we probably should try and emit a warning if people include it.
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.
See #4011.
Done. |
704271b
to
9a7e037
Compare
Retest this please. |
Retest this please |
Retest this please. |
1 similar comment
Retest this please. |
Deprecated code updates corresponding to kokkos/kokkos#3978
Deprecated code updates corresponding to kokkos/kokkos#3978
Deprecated code updates corresponding to kokkos/kokkos#3978
Deprecated code updates corresponding to kokkos/kokkos#3978
Deprecated code updates corresponding to kokkos/kokkos#3978
Deprecated code updates corresponding to kokkos/kokkos#3978
In the spirit of #3973. I decided not to touch
ViewAllocateWithoutInitializing
since we already have some kind of deprecation commented but not enabled.Deprecated:
Kokkos::Experimental::Iterate
->Kokkos::Iterate
Kokkos::Experimental::MDRangePolicy
->Kokkos::MDRangePolicy
Kokkos::Experimental::Iterate
->Kokkos::Iterate
Kokkos::Experimental::Rank
->Kokkos::Rank
Kokkos::Impl::is_array_layout
->Kokkos::is_array_layout
Kokkos::Impl::is_execution_policy
->Kokkos::is_execution_policy
Kokkos::Impl::is_execution_space
->Kokkos::is_execution_space
Kokkos::Impl::is_memory_space
->Kokkos::is_memory_space
Kokkos::Impl::is_memory_traits
->Kokkos::is_memory_traits
Kokkos::Impl::is_space
->Kokkos::is_space
Kokkos::Impl::SpaceAccessibility
->Kokkos::SpaceAccessibility