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

Replace default, constexpr, and delete macros by original keywords #360

Merged
merged 2 commits into from
Aug 23, 2023

Conversation

Blonck
Copy link
Contributor

@Blonck Blonck commented Aug 21, 2023

As part of #127, code for unsupported compilation modes/platforms will be removed. default, constexpr, and delete are supported by all targeted compilers so that the conditional support can be removed.

All _LIBCUDACXX_CONSTEXPR macros are replaced by constexpr. All _LIBCUDACXX_EQUAL_DEFAULT macros are replaced by = default;. All _LIBCUDACXX_EQUAL_DELETE macros are replaced by = delete;.

Checklist

@Blonck Blonck requested review from a team as code owners August 21, 2023 12:33
@Blonck Blonck requested review from ericniebler and alliepiper and removed request for a team August 21, 2023 12:33
@rapids-bot
Copy link

rapids-bot bot commented Aug 21, 2023

Pull requests from external contributors require approval from a NVIDIA organization member with write permissions or greater before CI can begin.

@miscco
Copy link
Collaborator

miscco commented Aug 21, 2023

/ok to test

1 similar comment
@miscco
Copy link
Collaborator

miscco commented Aug 22, 2023

/ok to test

Martin Marenz added 2 commits August 22, 2023 20:24
…macros

As part of NVIDIA#127, code for unsupported compilation modes/platforms will be removed.
`default`, `constexpr`, and `delete` are supported by all targeted compilers, so the conditional support can be removed.

All `_LIBCUDACXX_CONSTEXPR` macros are replaced by `constexpr`.
All `_LIBCUDACXX_EQUAL_DEFAULT` macros are replaced by `= default;`.
All `_LIBCUDACXX_EQUAL_DELETE` macros are replaced by `= delete;`.
@miscco
Copy link
Collaborator

miscco commented Aug 22, 2023

/ok to test

@miscco miscco merged commit 8197f8e into NVIDIA:main Aug 23, 2023
435 checks passed
@Blonck Blonck deleted the remove_cpp03_macros branch September 6, 2023 04:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants