Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Alias deprecated TexRefInputIterator to TexObjInputIterator. #451

Merged
merged 1 commit into from
Apr 11, 2022

Conversation

alliepiper
Copy link
Collaborator

Removes usage of deprecated CUDART texture reference APIs without
breaking CUB's API.

Other changes:

  • Remove obsolete checks for CUDART_VERSION >= 5.5.
  • Split test_iterator.cu to create a test that only handles the
    deprecated TexRefInputIterator API.
    • Reduces the scope of our deprecation suppressions.
    • Also removed the deprecation suppression from our CMake logic.
  • Enable testing for TexObjInputIterator without CUB_CDP.
  • Add a meaningful implementation for ostream << TexObjInputIterator.

alliepiper added a commit to alliepiper/thrust that referenced this pull request Mar 29, 2022
@alliepiper alliepiper added P0: must have Absolutely necessary. Critical issue, major blocker, etc. only: gpuci Changes to gpuCI only. Doesn't need internal NVIDIA CI. labels Mar 29, 2022
@alliepiper alliepiper added this to the 1.17.0 milestone Mar 29, 2022
@alliepiper
Copy link
Collaborator Author

gpuCI: NVIDIA/thrust#1652

alliepiper added a commit to alliepiper/thrust that referenced this pull request Mar 29, 2022
@alliepiper alliepiper added the testing: gpuCI passed Passed gpuCI testing. label Mar 30, 2022
alliepiper added a commit to alliepiper/thrust that referenced this pull request Apr 6, 2022
Removes usage of deprecated CUDART texture reference APIs without
breaking CUB's API.

Other changes:
- Remove obsolete checks for CUDART_VERSION >= 5.5.
- Split `test_iterator.cu` to create a test that only handles the
  deprecated `TexRefInputIterator` API.
  - Reduces the scope of our deprecation suppressions.
  - Also removed the deprecation suppression from our CMake logic.
- Enable testing for `TexObjInputIterator` without `CUB_CDP`.
- Add a meaningful implementation for `ostream << TexObjInputIterator`.
alliepiper added a commit to alliepiper/thrust that referenced this pull request Apr 6, 2022
@alliepiper alliepiper added testing: gpuCI in progress Started gpuCI testing. and removed testing: gpuCI passed Passed gpuCI testing. labels Apr 6, 2022
@alliepiper alliepiper merged commit ab5ee72 into NVIDIA:main Apr 11, 2022
@alliepiper alliepiper deleted the depr_tex_ref branch April 11, 2022 18:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
only: gpuci Changes to gpuCI only. Doesn't need internal NVIDIA CI. P0: must have Absolutely necessary. Critical issue, major blocker, etc. testing: gpuCI in progress Started gpuCI testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants