eyalroz / cuda-kat

CUDA kernel author's tools
BSD 3-Clause "New" or "Revised" License
104 stars 8 forks source link

Use std::forward and std::move, uniformly #59

Closed eyalroz closed 4 years ago

eyalroz commented 4 years ago

This is two issues in one:

  1. In some files (e.g. tuple.hpp), we are inconsistent in the use of kat::move() and kat::forward() vs std::move() and std::forward().
  2. The kat versions of these functions are not necessary, as nVIDIA makes an exception for their std versions and interprets them as __host__ __device__ functions.

So, let's just switch everything to the std versions. Howerver, we'll leae the kat versions within an #ifdef guard: KAT_DEFINE_MOVE_AND_FORWARD.