Skip to content
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 deprecation warning for AllocationMechanism for gcc <11.0 #6653

Conversation

masterleinad
Copy link
Contributor

Related to the conversation in https://kokkosteam.slack.com/archives/C5BGU5NDQ/p1701727560890859.
We see deprecation warnings for GCC versions before release 11 every time a compilation unit includes Kokkos_HostSpace.hpp.
This pull request proposes removing the deprecation attribute for AllocationMechanism to avoid these warnings. It seems very unlikely that the enum would be used without the HostSpace constructor in question that still has the deprecation attribute.

Copy link
Member

@dalg24 dalg24 left a comment

Choose a reason for hiding this comment

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

In case it is useful to anyone else https://godbolt.org/z/bbenczfbE

@dalg24
Copy link
Member

dalg24 commented Dec 7, 2023

Will need to cherry-pick into the patch release 4.2.01

Copy link
Contributor

@cz4rs cz4rs left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@dalg24 dalg24 merged commit 24b6484 into kokkos:develop Dec 7, 2023
28 of 29 checks passed
masterleinad pushed a commit that referenced this pull request Dec 7, 2023
…ion_mechanism_gcc_11_0

Remove deprecation warning for AllocationMechanism for gcc <11.0
cgcgcg added a commit to cgcgcg/Trilinos that referenced this pull request Dec 21, 2023
Propagates the change from

kokkos/kokkos#6653
cgcgcg added a commit to cgcgcg/Trilinos that referenced this pull request Dec 21, 2023
Propagates the change from

kokkos/kokkos#6653
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants