-
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 KOKKOS_THREAD_LOCAL #5064
Remove KOKKOS_THREAD_LOCAL #5064
Conversation
How sure are we that no one downstream uses that? Maybe deprecate for 3.7 and then remove (could define it to KOKKOS_THREAD_LOCAL = thread_local? |
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 we deprecate? Its a kinda public name?
OK with what Christian suggest. |
It's not used in |
4d1ac27
to
4973f49
Compare
Not sure what is going on with Jenkins but that commit was built successfully so I am going to merge as is |
There doesn't really seem any need to define this macro when we can use
thread_local
instead.