-
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
Reuse scratch memory in SYCL TeamPolicy #3889
Conversation
The whole implementation is pretty much taken from the |
11461d3
to
140418b
Compare
Plz rebase |
140418b
to
e22fd64
Compare
Done. |
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.
Looks good other than the header includes
@@ -42,6 +42,7 @@ | |||
//@HEADER | |||
*/ | |||
|
|||
#include <Kokkos_Core.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.
Why are you including <Kokkos_Core.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.
This is for kokkos_malloc
. I could try forward declaring in Kokkos_core_fwd.hpp
instead.
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.
It just makes little sense to include everything and below include selected headers.
c1cfd5c
to
ea6479e
Compare
Similar to #3873, we can reuse the scratch memory allocated for
TeamPolicy
.