Constrain PortableCollection and PortableObject CopyToDevice copies

In order to use requires expressions to check the existence of the
CopyToDevice specializations (because PortableDevice* types are
intended only for non-CPU device types).

Author: makortel

DataFormats/Portable/interface/PortableCollection.h

@@ -5,6 +5,7 @@
 
 #include "DataFormats/Portable/interface/PortableHostCollection.h"
 #include "DataFormats/Portable/interface/PortableDeviceCollection.h"
+#include "HeterogeneousCore/AlpakaInterface/interface/concepts.h"
 #include "HeterogeneousCore/AlpakaInterface/interface/CopyToDevice.h"
 #include "HeterogeneousCore/AlpakaInterface/interface/CopyToHost.h"
 
@@ -47,8 +48,10 @@ using PortableMultiCollection = typename traits::PortableMultiCollectionTrait<TD
 // define how to copy PortableCollection between host and device
 namespace cms::alpakatools {
   template <typename TLayout, typename TDevice>
+    requires alpaka::isDevice<TDevice>
   struct CopyToHost<PortableDeviceCollection<TLayout, TDevice>> {
     template <typename TQueue>
+      requires alpaka::isQueue<TQueue>
     static auto copyAsync(TQueue& queue, PortableDeviceCollection<TLayout, TDevice> const& srcData) {
       PortableHostCollection<TLayout> dstData(srcData->metadata().size(), queue);
       alpaka::memcpy(queue, dstData.buffer(), srcData.buffer());
@@ -57,8 +60,10 @@ namespace cms::alpakatools {
   };
 
   template <typename TDev, typename T0, typename... Args>
+    requires alpaka::isDevice<TDev>
   struct CopyToHost<PortableDeviceMultiCollection<TDev, T0, Args...>> {
     template <typename TQueue>
+      requires alpaka::isQueue<TQueue>
     static auto copyAsync(TQueue& queue, PortableDeviceMultiCollection<TDev, T0, Args...> const& srcData) {
       PortableHostMultiCollection<T0, Args...> dstData(srcData.sizes(), queue);
       alpaka::memcpy(queue, dstData.buffer(), srcData.buffer());
@@ -68,7 +73,7 @@ namespace cms::alpakatools {
 
   template <typename TLayout>
   struct CopyToDevice<PortableHostCollection<TLayout>> {
-    template <typename TQueue>
+    template <cms::alpakatools::NonCPUQueue TQueue>
     static auto copyAsync(TQueue& queue, PortableHostCollection<TLayout> const& srcData) {
       using TDevice = typename alpaka::trait::DevType<TQueue>::type;
       PortableDeviceCollection<TLayout, TDevice> dstData(srcData->metadata().size(), queue);
@@ -79,7 +84,7 @@ namespace cms::alpakatools {
 
   template <typename T0, typename... Args>
   struct CopyToDevice<PortableHostMultiCollection<T0, Args...>> {
-    template <typename TQueue>
+    template <cms::alpakatools::NonCPUQueue TQueue>
     static auto copyAsync(TQueue& queue, PortableHostMultiCollection<T0, Args...> const& srcData) {
       using TDevice = typename alpaka::trait::DevType<TQueue>::type;
       PortableDeviceMultiCollection<TDevice, T0, Args...> dstData(srcData.sizes(), queue);

DataFormats/Portable/interface/PortableObject.h

@@ -7,6 +7,7 @@
 
 #include "DataFormats/Portable/interface/PortableHostObject.h"
 #include "DataFormats/Portable/interface/PortableDeviceObject.h"
+#include "HeterogeneousCore/AlpakaInterface/interface/concepts.h"
 #include "HeterogeneousCore/AlpakaInterface/interface/CopyToDevice.h"
 #include "HeterogeneousCore/AlpakaInterface/interface/CopyToHost.h"
 
@@ -33,8 +34,10 @@ using PortableObject = typename traits::PortableObjectTrait<T, TDev>::ProductTyp
 // define how to copy PortableObject between host and device
 namespace cms::alpakatools {
   template <typename TProduct, typename TDevice>
+    requires alpaka::isDevice<TDevice>
   struct CopyToHost<PortableDeviceObject<TProduct, TDevice>> {
     template <typename TQueue>
+      requires alpaka::isQueue<TQueue>
     static auto copyAsync(TQueue& queue, PortableDeviceObject<TProduct, TDevice> const& srcData) {
       PortableHostObject<TProduct> dstData(queue);
       alpaka::memcpy(queue, dstData.buffer(), srcData.buffer());
@@ -44,7 +47,7 @@ namespace cms::alpakatools {
 
   template <typename TProduct>
   struct CopyToDevice<PortableHostObject<TProduct>> {
-    template <typename TQueue>
+    template <cms::alpakatools::NonCPUQueue TQueue>
     static auto copyAsync(TQueue& queue, PortableHostObject<TProduct> const& srcData) {
       using TDevice = typename alpaka::trait::DevType<TQueue>::type;
       PortableDeviceObject<TProduct, TDevice> dstData(queue);

HeterogeneousCore/AlpakaInterface/interface/concepts.h

@@ -0,0 +1,13 @@
+#ifndef HeterogeneousCore_AlpakaInterface_interface_concepts_h
+#define HeterogeneousCore_AlpakaInterface_interface_concepts_h
+
+#include <type_traits>
+
+#include <alpaka/alpaka.hpp>
+
+namespace cms::alpakatools {
+  template <typename T>
+  concept NonCPUQueue = alpaka::isQueue<T> and not std::is_same_v<alpaka::Dev<T>, alpaka::DevCpu>;
+}  // namespace cms::alpakatools
+
+#endif