Merge pull request #1793 from arcaneframework/dev/gg-use-viewbuildinfo-for-material-view

Use 'ViewBuildInfo' instead of 'RunCommand' to build material views on accelerator

Author: grospelliergilles

arcane/src/arcane/accelerator/MaterialVariableViews.cc

@@ -0,0 +1,44 @@
+// -*- tab-width: 2; indent-tabs-mode: nil; coding: utf-8-with-signature -*-
+//-----------------------------------------------------------------------------
+// Copyright 2000-2024 CEA (www.cea.fr) IFPEN (www.ifpenergiesnouvelles.com)
+// See the top-level COPYRIGHT file for details.
+// SPDX-License-Identifier: Apache-2.0
+//-----------------------------------------------------------------------------
+/*---------------------------------------------------------------------------*/
+/* MaterialVariableViews.cc                                    (C) 2000-2024 */
+/*                                                                           */
+/* Gestion des vues sur les variables matériaux pour les accélérateurs.      */
+/*---------------------------------------------------------------------------*/
+/*---------------------------------------------------------------------------*/
+
+#include "arcane/accelerator/MaterialVariableViews.h"
+
+/*---------------------------------------------------------------------------*/
+/*---------------------------------------------------------------------------*/
+/*!
+ * \file MaterialVariableViews.h
+ *
+ * Ce fichier contient les déclarations des types pour gérer
+ * les vues pour les accélérateurs des variables du maillage.
+ */
+/*---------------------------------------------------------------------------*/
+/*---------------------------------------------------------------------------*/
+
+namespace Arcane::Accelerator
+{
+
+/*---------------------------------------------------------------------------*/
+/*---------------------------------------------------------------------------*/
+
+MatVariableViewBase::
+MatVariableViewBase(const ViewBuildInfo&, IMeshMaterialVariable*)
+{
+}
+
+/*---------------------------------------------------------------------------*/
+/*---------------------------------------------------------------------------*/
+
+} // namespace Arcane::Accelerator
+
+/*---------------------------------------------------------------------------*/
+/*---------------------------------------------------------------------------*/

arcane/src/arcane/accelerator/MaterialVariableViews.h

@@ -21,6 +21,8 @@
 #include "arcane/core/materials/MeshEnvironmentVariableRef.h"
 #include "arcane/core/materials/MatItem.h"
 
+#include "arcane/accelerator/AcceleratorGlobal.h"
+
 /*---------------------------------------------------------------------------*/
 /*---------------------------------------------------------------------------*/
 
@@ -38,34 +40,35 @@ namespace Arcane::Accelerator
 /*!
  * \brief Classe de base des vues sur les variables matériaux.
  */
-class MatVariableViewBase
+class ARCANE_ACCELERATOR_EXPORT MatVariableViewBase
 {
  public:
+
   // Pour l'instant n'utilise pas encore les paramètres
-  MatVariableViewBase(RunCommand&, IMeshMaterialVariable*)
-  {
-  }
- private:
+  MatVariableViewBase(const ViewBuildInfo&, IMeshMaterialVariable*);
 };
 
 /*---------------------------------------------------------------------------*/
 /*---------------------------------------------------------------------------*/
 /*!
  * \brief Vue en lecture sur une variable scalaire du maillage.
- * TODO: Faut-il rajouter la gestion des SIMD comme dans ItemVariableScalarInViewT ?
  */
-template<typename ItemType,typename DataType>
+template <typename ItemType, typename DataType>
 class MatItemVariableScalarInViewT
 : public MatVariableViewBase
 {
+  // TODO: Faut-il rajouter la gestion des SIMD comme dans ItemVariableScalarInViewT ?
+
  private:
 
   using ItemIndexType = typename ItemTraitsT<ItemType>::LocalIdType;
 
  public:
 
-  MatItemVariableScalarInViewT(RunCommand& cmd, IMeshMaterialVariable* var, ArrayView<DataType>* v)
-  : MatVariableViewBase(cmd, var), m_value(v){}
+  MatItemVariableScalarInViewT(const ViewBuildInfo& vbi, IMeshMaterialVariable* var, ArrayView<DataType>* v)
+  : MatVariableViewBase(vbi, var)
+  , m_value(v)
+  {}
 
   //! Opérateur d'accès pour l'entité \a item
   ARCCORE_HOST_DEVICE const DataType& operator[](ComponentItemLocalId lid) const
@@ -101,26 +104,12 @@ class MatItemVariableScalarInViewT
   ArrayView<DataType>* m_value;
 };
 
-/*---------------------------------------------------------------------------*/
-/*---------------------------------------------------------------------------*/
-
-/*
- * TODO: A faire ?
- *
-template<typename ItemType,typename DataType>
-class MatItemVariableArrayInViewT
-: public MatVariableViewBase
-{
-};
- */
-
 /*---------------------------------------------------------------------------*/
 /*---------------------------------------------------------------------------*/
 /*!
  * \brief Vue en écriture sur une variable scalaire du maillage.
- * TODO: Faut il rajouter la gestion des types SIMD comme dans ItemVariableScalarOutViewT ?
  */
-template<typename ItemType,typename Accessor>
+template <typename ItemType, typename Accessor>
 class MatItemVariableScalarOutViewT
 : public MatVariableViewBase
 {
@@ -130,17 +119,21 @@ class MatItemVariableScalarOutViewT
   using DataTypeReturnType = DataType&;
   using ItemIndexType = typename ItemTraitsT<ItemType>::LocalIdType;
 
-  // TODO: faut il rajouter des ARCANE_CHECK_AT(mvi.arrayIndex(), m_value.size()); ? il manquera tjrs le check sur l'autre dimension
+  // TODO: faut il rajouter des ARCANE_CHECK_AT(mvi.arrayIndex(), m_value.size());
+  // il manquera tjrs le check sur l'autre dimension
+  // TODO: Faut il rajouter la gestion des types SIMD comme dans ItemVariableScalarOutViewT ?
 
  public:
 
-  MatItemVariableScalarOutViewT(RunCommand& cmd,IMeshMaterialVariable* var,ArrayView<DataType>* v)
-  : MatVariableViewBase(cmd, var), m_value(v){}
+  MatItemVariableScalarOutViewT(const ViewBuildInfo& vbi, IMeshMaterialVariable* var, ArrayView<DataType>* v)
+  : MatVariableViewBase(vbi, var)
+  , m_value(v)
+  {}
 
   //! Opérateur d'accès pour l'entité \a item
   ARCCORE_HOST_DEVICE Accessor operator[](ComponentItemLocalId lid) const
   {
-    return Accessor(this->m_value[lid.localId().arrayIndex()].data()+lid.localId().valueIndex());
+    return Accessor(this->m_value[lid.localId().arrayIndex()].data() + lid.localId().valueIndex());
   }
 
   ARCCORE_HOST_DEVICE Accessor operator[](PureMatVarIndex pmvi) const
@@ -151,18 +144,18 @@ class MatItemVariableScalarOutViewT
   //! Surcharge pour accéder à la valeure globale à partir du cell id
   ARCCORE_HOST_DEVICE Accessor operator[](ItemIndexType item) const
   {
-    ARCANE_CHECK_AT(item.localId(),this->m_value[0].size());
+    ARCANE_CHECK_AT(item.localId(), this->m_value[0].size());
     return Accessor(this->m_value[0].data() + item.localId());
   }
 
   //! Opérateur d'accès pour l'entité \a item
   ARCCORE_HOST_DEVICE Accessor value(ComponentItemLocalId lid) const
   {
-    return Accessor(this->m_value[lid.localId().arrayIndex()].data()+lid.localId().valueIndex());
+    return Accessor(this->m_value[lid.localId().arrayIndex()].data() + lid.localId().valueIndex());
   }
 
   //! Positionne la valeur pour l'entité \a item à \a v
-  ARCCORE_HOST_DEVICE void setValue(ComponentItemLocalId lid,const DataType& v) const
+  ARCCORE_HOST_DEVICE void setValue(ComponentItemLocalId lid, const DataType& v) const
   {
     this->m_value[lid.localId().arrayIndex()][lid.localId().valueIndex()] = v;
   }
@@ -182,47 +175,47 @@ class MatItemVariableScalarOutViewT
 /*!
  * \brief Vue en écriture pour les variables materiaux scalaire
  */
-template<typename DataType> auto
-viewOut(RunCommand& cmd, CellMaterialVariableScalarRef<DataType>& var)
+template <typename DataType> auto
+viewOut(const ViewBuildInfo& vbi, CellMaterialVariableScalarRef<DataType>& var)
 {
   using Accessor = DataViewSetter<DataType>;
-  return MatItemVariableScalarOutViewT<Cell,Accessor>(cmd, var.materialVariable(),var._internalValue());
+  return MatItemVariableScalarOutViewT<Cell, Accessor>(vbi, var.materialVariable(), var._internalValue());
 }
 
 /*---------------------------------------------------------------------------*/
 /*---------------------------------------------------------------------------*/
 /*!
  * \brief Vue en écriture pour les variables materiaux scalaire
  */
-template<typename DataType> auto
-viewOut(RunCommand& cmd, CellEnvironmentVariableScalarRef<DataType>& var)
+template <typename DataType> auto
+viewOut(const ViewBuildInfo& vbi, CellEnvironmentVariableScalarRef<DataType>& var)
 {
   using Accessor = DataViewSetter<DataType>;
-  return MatItemVariableScalarOutViewT<Cell,Accessor>(cmd, var.materialVariable(),var._internalValue());
+  return MatItemVariableScalarOutViewT<Cell, Accessor>(vbi, var.materialVariable(), var._internalValue());
 }
 
 /*---------------------------------------------------------------------------*/
 /*---------------------------------------------------------------------------*/
 /*!
  * \brief Vue en lecture/écriture pour les variables materiaux scalaire
  */
-template<typename DataType> auto
-viewInOut(RunCommand& cmd, CellMaterialVariableScalarRef<DataType>& var)
+template <typename DataType> auto
+viewInOut(const ViewBuildInfo& vbi, CellMaterialVariableScalarRef<DataType>& var)
 {
   using Accessor = DataViewGetterSetter<DataType>;
-  return MatItemVariableScalarOutViewT<Cell,Accessor>(cmd, var.materialVariable(),var._internalValue());
+  return MatItemVariableScalarOutViewT<Cell, Accessor>(vbi, var.materialVariable(), var._internalValue());
 }
 
 /*---------------------------------------------------------------------------*/
 /*---------------------------------------------------------------------------*/
 /*!
  * \brief Vue en lecture/écriture pour les variables materiaux scalaire
  */
-template<typename DataType> auto
-viewInOut(RunCommand& cmd, CellEnvironmentVariableScalarRef<DataType>& var)
+template <typename DataType> auto
+viewInOut(const ViewBuildInfo& vbi, CellEnvironmentVariableScalarRef<DataType>& var)
 {
   using Accessor = DataViewGetterSetter<DataType>;
-  return MatItemVariableScalarOutViewT<Cell,Accessor>(cmd, var.materialVariable(),var._internalValue());
+  return MatItemVariableScalarOutViewT<Cell, Accessor>(vbi, var.materialVariable(), var._internalValue());
 }
 
 /*---------------------------------------------------------------------------*/
@@ -231,10 +224,10 @@ viewInOut(RunCommand& cmd, CellEnvironmentVariableScalarRef<DataType>& var)
 /*!
  * \brief Vue en lecture pour les variables materiaux scalaire
  */
-template<typename DataType> auto
-viewIn(RunCommand& cmd,const CellMaterialVariableScalarRef<DataType>& var)
+template <typename DataType> auto
+viewIn(const ViewBuildInfo& vbi, const CellMaterialVariableScalarRef<DataType>& var)
 {
-  return MatItemVariableScalarInViewT<Cell,DataType>(cmd, var.materialVariable(),var._internalValue());
+  return MatItemVariableScalarInViewT<Cell, DataType>(vbi, var.materialVariable(), var._internalValue());
 }
 
 /*---------------------------------------------------------------------------*/
@@ -243,15 +236,15 @@ viewIn(RunCommand& cmd,const CellMaterialVariableScalarRef<DataType>& var)
 /*!
  * \brief Vue en lecture pour les variables materiaux scalaire
  */
-template<typename DataType> auto
-viewIn(RunCommand& cmd,const CellEnvironmentVariableScalarRef<DataType>& var)
+template <typename DataType> auto
+viewIn(const ViewBuildInfo& vbi, const CellEnvironmentVariableScalarRef<DataType>& var)
 {
-  return MatItemVariableScalarInViewT<Cell,DataType>(cmd, var.materialVariable(),var._internalValue());
+  return MatItemVariableScalarInViewT<Cell, DataType>(vbi, var.materialVariable(), var._internalValue());
 }
 
 /*---------------------------------------------------------------------------*/
 /*---------------------------------------------------------------------------*/
 
-}
+} // namespace Arcane::Accelerator
 
 #endif

arcane/src/arcane/accelerator/VariableViews.h

@@ -22,6 +22,7 @@
 #include "arcane/core/VariableTypedef.h"
 #include "arcane/core/GroupIndexTable.h"
 
+#include "arcane/accelerator/core/ViewBuildInfo.h"
 #include "arcane/accelerator/AcceleratorGlobal.h"
 #include "arcane/accelerator/ViewsCommon.h"
 

arcane/src/arcane/accelerator/srcs.cmake

@@ -16,6 +16,7 @@ set( ARCANE_SOURCES
   GenericSorter.h
   IReduceMemoryImpl.h
   MaterialVariableViews.h
+  MaterialVariableViews.cc
   MemoryCopier.cc
   NumArray.h
   NumArrayViews.h