Update deprecation mechanism (opt-out, cmake, c++14).

Opt-out for our internal builds, since we'll still need to maintain and test any deprecated APIs.
NVIDIA · Jun 9, 2021 · 27518ae · 27518ae
1 parent 7736317
commit 27518ae
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 2 deletions.
diff --git a/cmake/ThrustBuildTargetList.cmake b/cmake/ThrustBuildTargetList.cmake
@@ -314,6 +314,10 @@ function(thrust_build_target_list)
   add_flag_option(IGNORE_DEPRECATED_COMPILER "Don't warn about deprecated compilers." OFF)
   add_flag_option(IGNORE_CUB_VERSION_CHECK "Don't warn about mismatched CUB versions." OFF)
 
+  # By default, suppress deprecation warnings when building our test suite,
+  ## since we'll need to test deprecated APIs with `-Werror`.
+  add_flag_option(IGNORE_DEPRECATED_API "Don't warn about deprecated Thrust or CUB APIs." ON)
+
   # Top level meta-target. Makes it easier to just build thrust targets when
   # building both CUB and Thrust. Add all project files here so IDEs will be
   # aware of them. This will not generate build rules.

diff --git a/dependencies/cub b/dependencies/cub
diff --git a/thrust/cmake/thrust-config.cmake b/thrust/cmake/thrust-config.cmake
@@ -50,6 +50,7 @@
 #
 # # Create target with HOST=CPP DEVICE=CUDA and some advanced flags set
 # thrust_create_target(TargetName
+#   IGNORE_DEPRECATED_API         # Silence build warnings about deprecated APIs
 #   IGNORE_DEPRECATED_CPP_DIALECT # Silence build warnings about deprecated compilers and C++ standards
 #   IGNORE_DEPRECATED_CPP_11      # Only silence deprecation warnings for C++11
 #   IGNORE_DEPRECATED_COMPILER    # Only silence deprecation warnings for old compilers
@@ -104,6 +105,7 @@ function(thrust_create_target target_name)
     ADVANCED
     FROM_OPTIONS
     IGNORE_CUB_VERSION_CHECK
+    IGNORE_DEPRECATED_API
     IGNORE_DEPRECATED_COMPILER
     IGNORE_DEPRECATED_CPP_11
     IGNORE_DEPRECATED_CPP_DIALECT
@@ -196,6 +198,10 @@ function(thrust_create_target target_name)
     target_compile_definitions(${target_name} INTERFACE "THRUST_IGNORE_DEPRECATED_CPP_DIALECT")
   endif()
 
+  if (TCT_IGNORE_DEPRECATED_API)
+    target_compile_definitions(${target_name} INTERFACE "THRUST_IGNORE_DEPRECATED_API")
+  endif()
+
   if (TCT_IGNORE_DEPRECATED_CPP_11)
     target_compile_definitions(${target_name} INTERFACE "THRUST_IGNORE_DEPRECATED_CPP_11")
   endif()

diff --git a/thrust/detail/config/deprecated.h b/thrust/detail/config/deprecated.h
@@ -21,8 +21,17 @@
 #pragma once
 
 #include <thrust/detail/config/compiler.h>
+#include <thrust/detail/config/cpp_dialect.h>
 
-#if THRUST_HOST_COMPILER == THRUST_HOST_COMPILER_MSVC
+#if defined(CUB_IGNORE_DEPRECATED_API) && !defined(THRUST_IGNORE_DEPRECATED_API)
+#  define THRUST_IGNORE_DEPRECATED_API
+#endif
+
+#ifdef THRUST_IGNORE_DEPRECATED_API
+#  define THRUST_DEPRECATED
+#elif THRUST_CPP_DIALECT >= 2014
+#  define THRUST_DEPRECATED [[deprecated]]
+#elif THRUST_HOST_COMPILER == THRUST_HOST_COMPILER_MSVC
 #  define THRUST_DEPRECATED __declspec(deprecated)
 #elif THRUST_HOST_COMPILER == THRUST_HOST_COMPILER_CLANG
 #  define THRUST_DEPRECATED __attribute__((deprecated))
+11 −1		CMakeLists.txt
+2 −2		cub/agent/agent_spmv_orig.cuh
+4 −0		cub/cmake/cub-config.cmake
+1 −1		cub/iterator/tex_obj_input_iterator.cuh
+46 −23		cub/iterator/tex_ref_input_iterator.cuh
+10 −2		cub/util_deprecated.cuh
+1 −1		test/test_iterator.cu