Skip to content

Comments

kokkos: bring back deprecated_code variant#3154

Open
rbberger wants to merge 2 commits intospack:developfrom
rbberger:updates/kokkos-deprecated
Open

kokkos: bring back deprecated_code variant#3154
rbberger wants to merge 2 commits intospack:developfrom
rbberger:updates/kokkos-deprecated

Conversation

@rbberger
Copy link
Member

@rbberger rbberger commented Jan 27, 2026

No description provided.

Copy link
Contributor

@tpadioleau tpadioleau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually the CMake defaults are to disable 4 and enable 5 in Kokkos 5, https://github.com/kokkos/kokkos/blob/a5d374ba179921b9805008dda06a4b690cfbdd1a/cmake/kokkos_enable_options.cmake#L41-L42. Note that the documentation seems wrong, https://kokkos.org/kokkos-core-wiki/get-started/configuration-guide.html#general-options.

I wonder if giving the control only over the current major version would be enough ? deprecated_code controls Kokkos_ENABLE_DEPRECATED_CODE_X for Kokkos@X ?

@rbberger
Copy link
Member Author

@tpadioleau I'll do whatever you prefer. It was my understanding that people were asking for X-1 deprecated code.

@rbberger
Copy link
Member Author

It would also be informative if Kokkos plans to always have both X and X-1 depreciated code

@rbberger rbberger force-pushed the updates/kokkos-deprecated branch from 7b9c1b5 to 0e73e22 Compare January 29, 2026 20:47
@spackbot-triage spackbot-triage bot requested a review from tpadioleau January 29, 2026 20:48
@rbberger
Copy link
Member Author

@tpadioleau changed the default to match the CMake defaults. I guess we wait until the next WG meeting to decide if we want the simpler variant?

@cedricchevalier19
Copy link
Contributor

During the WG meeting we say we want to keep the logic simple, so using a boolean variant is the group choice.

@rbberger rbberger force-pushed the updates/kokkos-deprecated branch 2 times, most recently from bf282d6 to f367b72 Compare February 18, 2026 03:10
@rbberger rbberger force-pushed the updates/kokkos-deprecated branch from f367b72 to 4a20bda Compare February 19, 2026 21:49
@spackbot-triage spackbot-triage bot requested a review from tpadioleau February 19, 2026 21:49
@spackbot-triage spackbot-triage bot requested a review from tpadioleau February 21, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants