OilfieldToolsNet/opm-simulators
4
0
Fork 0
mirror of https://github.com/OPM/opm-simulators.git synced 2025-02-25 18:55:30 -06:00
Code Issues Packages Projects Releases Wiki Activity

fixed: don't add device/host annotations to defaulted functions

Browse Source 
these will automagically be generated in both compiler passes so
the specifiers are meaningless and causes nvcc to emit a warning
This commit is contained in:
Arne Morten Kvarving 2024-08-07 09:54:59 +02:00
parent b145b6bf11
commit 42389b0182
1 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
opm/simulators/linalg/cuistl/CuView.hpp
View File

@ -18,14 +18,17 @@
*/ */
#ifndef OPM_CUVIEW_HEADER_HPP #ifndef OPM_CUVIEW_HEADER_HPP
#define OPM_CUVIEW_HEADER_HPP #define OPM_CUVIEW_HEADER_HPP
#include <dune/common/fvector.hh> #include <dune/common/fvector.hh>
#include <dune/istl/bvector.hh>
#include <exception>
#include <fmt/core.h>
#include <opm/common/ErrorMacros.hpp> #include <opm/common/ErrorMacros.hpp>
#include <opm/simulators/linalg/cuistl/detail/safe_conversion.hpp> #include <opm/simulators/linalg/cuistl/detail/safe_conversion.hpp>
#include <stdexcept>
#include <vector> #include <vector>
#include <string>
#include <fmt/core.h>
// TODO: remove this line and instead include gpuDecorators.hpp from OPM common when it gets added // TODO: remove this line and instead include gpuDecorators.hpp from OPM common when it gets added
#if defined(__CUDA_ARCH__) || (defined(__HIP_DEVICE_COMPILE__) && __HIP_DEVICE_COMPILE__ > 0) #if defined(__CUDA_ARCH__) || (defined(__HIP_DEVICE_COMPILE__) && __HIP_DEVICE_COMPILE__ > 0)
@ -34,8 +37,6 @@
#define OPM_IS_INSIDE_DEVICE_FUNCTION_TEMPORARY 0 #define OPM_IS_INSIDE_DEVICE_FUNCTION_TEMPORARY 0
#endif #endif
namespace Opm::cuistl namespace Opm::cuistl
{ {
@ -61,7 +62,7 @@ public:
/** /**
* @brief Default constructor that will initialize cublas and allocate 0 bytes of memory * @brief Default constructor that will initialize cublas and allocate 0 bytes of memory
*/ */
__host__ __device__ explicit CuView() = default; explicit CuView() = default;
//TODO: we probably dont need anything like this or is it useful to have views also be able to handle things on CPU? //TODO: we probably dont need anything like this or is it useful to have views also be able to handle things on CPU?
/// @brief constructor based on std::vectors, this will make a view on the CPU /// @brief constructor based on std::vectors, this will make a view on the CPU
@ -110,7 +111,7 @@ public:
/** /**
* @brief ~CuView calls cudaFree * @brief ~CuView calls cudaFree
*/ */
__host__ __device__ ~CuView() = default; ~CuView() = default;
/** /**
* @return the raw pointer to the GPU data * @return the raw pointer to the GPU data
@ -410,4 +411,5 @@ private:
}; };
} // namespace Opm::cuistl } // namespace Opm::cuistl
#endif #endif