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

Deprecate TexRefInputIterator; underlying CUDA APIs are deprecated. #191

Closed
alliepiper opened this issue Jul 6, 2020 · 0 comments · Fixed by #320
Closed

Deprecate TexRefInputIterator; underlying CUDA APIs are deprecated. #191

alliepiper opened this issue Jul 6, 2020 · 0 comments · Fixed by #320
Assignees
Labels
P0: must have Absolutely necessary. Critical issue, major blocker, etc. type: bug: functional Does not work as intended.
Milestone

Comments

@alliepiper
Copy link
Collaborator

These will need to be refactored out at some point.

alliepiper referenced this issue in brycelelbach/cub_historical_2019_2020 Jul 9, 2020
- Deprecation warnings are disabled due to use of deprecated CUDA APIs
  such as cudaBindTexture. Filed NVlabs/cub#191 to track this.
- Bump CMake minimum version to 3.15.
- Remove workarounds for old cmake/compilers.
- Switch to CUB_ENABLE_DIALECT_CPPXX instead of CMAKE_CXX_STANDARD.
  - Like Thrust multiconfig, all can be enabled at once.
  - Uses Thrust's settings when Thrust multiconfig is enabled.
- Add CUB_ENABLE_* options to disable portions of the build.
  - HEADER_TESTING, TESTING, EXAMPLES
- Add sm_37, sm_80, remove sm_30, sm_32.
  - When building CUB in Thrust, just use thrust's flags.
- Sanitize variable names in AppendOptionIfAvailable.
  - GCC was failing to detect `-Werror`. Long story short, this fixes it.
- Add cub.compiler_interface target to store warning suppressions, etc.
- Find and use Thrust. We should move the Thrust code into Thrust.
- Check for THOROUGH, QUICK and QUICKER variants of tests, create new
  targets for them when found.
- Remove `examples/block/reduce_by_key.cu`, since it doesn't build and is
  incomplete.
- Disable NPP code paths in test_device_histogram as they don't compile.
  Seems like a Dispatch<int, int, NPP> specialization got removed at some
  point.
- Misc warning cleanups.
- Add all project files to custom target for IDE detection.
@alliepiper alliepiper added the type: bug: functional Does not work as intended. label Oct 21, 2020
@alliepiper alliepiper changed the title CUB uses deprecated APIs (cudaBindTexture) Deprecate TexRefInputIterator; underlying CUDA APIs are deprecated. Dec 10, 2020
@alliepiper alliepiper added this to the 1.12.0 milestone Dec 10, 2020
alliepiper added a commit to alliepiper/cub that referenced this issue Dec 10, 2020
We need to deprecate this class since the underlying CUDA APIs
are deprecated. This suppression is a temporary workaround.

Tracked by NVIDIA#191.
alliepiper added a commit to alliepiper/cub that referenced this issue Dec 11, 2020
We need to deprecate this class since the underlying CUDA APIs
are deprecated. This suppression is a temporary workaround.

Tracked by NVIDIA#191.
alliepiper added a commit to alliepiper/cub that referenced this issue Dec 11, 2020
We need to deprecate this class since the underlying CUDA APIs
are deprecated. This suppression is a temporary workaround.

Tracked by NVIDIA#191.
alliepiper added a commit to alliepiper/cub that referenced this issue Jan 22, 2021
We need to deprecate this class since the underlying CUDA APIs
are deprecated. This suppression is a temporary workaround.

Tracked by NVIDIA#191.
@alliepiper alliepiper modified the milestones: 1.12.0, 1.13.0 Feb 8, 2021
alliepiper added a commit to alliepiper/cub that referenced this issue Feb 10, 2021
We need to deprecate this class since the underlying CUDA APIs
are deprecated. This suppression is a temporary workaround.

Tracked by NVIDIA#191.
@alliepiper alliepiper self-assigned this Jun 9, 2021
@alliepiper alliepiper added the P0: must have Absolutely necessary. Critical issue, major blocker, etc. label Jun 9, 2021
@alliepiper alliepiper linked a pull request Jun 9, 2021 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
P0: must have Absolutely necessary. Critical issue, major blocker, etc. type: bug: functional Does not work as intended.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant