-
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 Kokkos_TaskPolicy.hpp #4011
Conversation
core/src/Kokkos_Macros.hpp
Outdated
#define KOKKOS_STRINGIZE_HELPER(x) #x | ||
#define KOKKOS_STRINGIZE(x) KOKKOS_STRINGIZE_HELPER(x) |
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.
kokkos/core/src/impl/Kokkos_Error.hpp
Lines 221 to 222 in 2265de3
#define KOKKOS_IMPL_STRINGIFY(x) #x | |
#define KOKKOS_IMPL_TOSTRING(x) KOKKOS_IMPL_STRINGIFY(x) |
core/src/Kokkos_Macros.hpp
Outdated
__FILE__ "(" KOKKOS_IMPL_TOSTRING(__LINE__) ") : warning: " #desc)) | ||
#else | ||
#define KOKKOS_DO_PRAGMA(x) _Pragma(#x) | ||
#define KOKKOS_WARNING(desc) KOKKOS_DO_PRAGMA(message(#desc)) |
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.
Would you remind me why this is it not just #warning <msg>
?
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.
That's what we are using in deal.II. Just #warning
wasn't portable enough to also work with MSVC
so we needed some kind of preprocessor macro and that's why we came up with this partly taken from http://goodliffe.blogspot.com/2009/07/c-how-to-say-warning-to-visual-studio-c.html. gcc
and clang
also don't like the MSVC
version. I'm happy to replace it if there are better suggestions.
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.
Should this be KOKKOS_IMPL_DO_PRAGMA and KOKKOS_IMPL_WARNING? I am nto sure I want to add them as public macros or?
Retest this please. |
Addressing #3978 (comment) we wanted to deprecate this header. Since it requires defining some more preprocessor macros (differing between architectures) I decided to split this up into its own pull request.