RAJA loop_exec deprecated for seq_exec (#309)

wrtobin · web-flow · commit d5b5bd88c27f · 2023-12-20T08:30:32.000-08:00
loop_exec-&gt;seq_exec
diff --git a/docs/sphinx/ArrayOfArrays.rst b/docs/sphinx/ArrayOfArrays.rst
@@ -226,8 +226,8 @@ The following timings are from a clang 10 release build on LLNL's Quartz system
 Function                 RAJA Policy               Time
 ======================== ========================= ======
 ``vector``               N/A                       0.99s
-``overAllocation``       ``loop_exec``             0.49s
-``resizeFromCapacities`` ``loop_exec``             0.58s
+``overAllocation``       ``seq_exec``             0.49s
+``resizeFromCapacities`` ``seq_exec``             0.58s
 ``overAllocation``       ``omp_parallel_for_exec`` 0.11s
 ``resizeFromCapacities`` ``omp_parallel_for_exec`` 0.17s
 ======================== ========================= ======
diff --git a/examples/exampleArray.cpp b/examples/exampleArray.cpp
@@ -350,7 +350,7 @@ CUDA_TEST( Array, chaiBuffer )
                       LvArray::ChaiBuffer > const & view = array;
 
   // Capture the view in a host kernel which moves the data back to the host.
-  RAJA::forall< RAJA::loop_exec >(
+  RAJA::forall< RAJA::seq_exec >(
     RAJA::TypedRangeSegment< std::ptrdiff_t >( 0, view.size() ),
     [view] ( std::ptrdiff_t const i )
   {
diff --git a/examples/exampleArrayOfArrays.cpp b/examples/exampleArrayOfArrays.cpp
@@ -266,7 +266,7 @@ TEST( ArrayOfArrays, resizeFromCapacities )
 
   // Resize the ArrayOfArrays from a new list of capacities.
   std::array< std::ptrdiff_t, 3 > newCapacities = { 3, 5, 2 };
-  arrayOfArrays.resizeFromCapacities< RAJA::loop_exec >( newCapacities.size(), newCapacities.data() );
+  arrayOfArrays.resizeFromCapacities< RAJA::seq_exec >( newCapacities.size(), newCapacities.data() );
 
   // This will clear any existing arrays.
   EXPECT_EQ( arrayOfArrays.size(), 3 );
@@ -311,7 +311,7 @@ CUDA_TEST( ArrayOfArrays, ChaiBuffer )
 
     // Capture the view on the host. This will copy back the values and sizes since they were previously touched
     // on device. It will only touch the values on host.
-    RAJA::forall< RAJA::loop_exec >(
+    RAJA::forall< RAJA::seq_exec >(
       RAJA::TypedRangeSegment< std::ptrdiff_t >( 0, viewConstSizes.size() ),
       [viewConstSizes] ( std::ptrdiff_t const i )
     {
@@ -332,7 +332,7 @@ CUDA_TEST( ArrayOfArrays, ChaiBuffer )
 
     // Capture the view on device. Since the values were previously touched on host it will copy them over.
     // Both the sizes and offsets are current on device so they are not copied over. Nothing is touched.
-    RAJA::forall< RAJA::loop_exec >(
+    RAJA::forall< RAJA::seq_exec >(
       RAJA::TypedRangeSegment< std::ptrdiff_t >( 0, viewConst.size() ),
       [viewConst] ( std::ptrdiff_t const i )
     {
diff --git a/examples/exampleArrayOfSets.cpp b/examples/exampleArrayOfSets.cpp
@@ -59,7 +59,7 @@ TEST( ArrayOfSets, assimilate )
   arrayOfArrays.emplaceBack( 2, 1 );
 
   // Assimilate arrayOfArrays into arrayOfSets.
-  arrayOfSets.assimilate< RAJA::loop_exec >( std::move( arrayOfArrays ),
+  arrayOfSets.assimilate< RAJA::seq_exec >( std::move( arrayOfArrays ),
                                              LvArray::sortedArrayManipulation::Description::SORTED_UNIQUE );
 
   // After being assimilated arrayOfArrays is empty.
@@ -84,7 +84,7 @@ TEST( ArrayOfSets, assimilate )
   arrayOfArrays.emplaceBack( 1, 4 );
 
   // Assimilate the arrayOfArrays yet again.
-  arrayOfSets.assimilate< RAJA::loop_exec >( std::move( arrayOfArrays ),
+  arrayOfSets.assimilate< RAJA::seq_exec >( std::move( arrayOfArrays ),
                                              LvArray::sortedArrayManipulation::Description::UNSORTED_WITH_DUPLICATES );
 
   EXPECT_EQ( arrayOfSets.size(), 2 );
diff --git a/examples/exampleBuffers.cpp b/examples/exampleBuffers.cpp
@@ -102,7 +102,7 @@ CUDA_TEST( ChaiBuffer, captureOnDevice )
   } );
 
   // Capture buffer in a host kernel moving the data back to the host allocation.
-  RAJA::forall< RAJA::loop_exec >(
+  RAJA::forall< RAJA::seq_exec >(
     RAJA::TypedRangeSegment< std::ptrdiff_t >( 0, size ),
     [buffer] ( std::ptrdiff_t const i )
   {
@@ -138,7 +138,7 @@ CUDA_TEST( ChaiBuffer, captureOnDeviceConst )
   // Capture buffer in a host kernel moving the data back to the host allocation.
   // If constBuffer didn't contain "int const" then this check would fail because
   // the data would be copied back from device.
-  RAJA::forall< RAJA::loop_exec >(
+  RAJA::forall< RAJA::seq_exec >(
     RAJA::TypedRangeSegment< std::ptrdiff_t >( 0, size ),
     [buffer] ( std::ptrdiff_t const i )
   {
diff --git a/examples/exampleSparsityPatternAndCRSMatrix.cpp b/examples/exampleSparsityPatternAndCRSMatrix.cpp
@@ -122,7 +122,7 @@ TEST( CRSMatrix, assimilate )
 
   // Create a matrix with the sparsity pattern
   LvArray::CRSMatrix< double, int, std::ptrdiff_t, LvArray::MallocBuffer > matrix;
-  matrix.assimilate< RAJA::loop_exec >( std::move( sparsity ) );
+  matrix.assimilate< RAJA::seq_exec >( std::move( sparsity ) );
 
   // The sparsity is empty after being assimilated.
   EXPECT_EQ( sparsity.numRows(), 0 );
diff --git a/src/Macros.hpp b/src/Macros.hpp
@@ -686,7 +686,7 @@ template< typename >
 struct RAJAHelper
 {};
 
-using serialPolicy = RAJA::loop_exec;
+using serialPolicy = RAJA::seq_exec;
 
 template<>
 struct RAJAHelper< serialPolicy >
diff --git a/src/python/PyCRSMatrix.hpp b/src/python/PyCRSMatrix.hpp
@@ -258,7 +258,7 @@ class PyCRSMatrixWrapper final : public PyCRSMatrixWrapperBase
     using EXEC_POLICY = RAJA::omp_parallel_for_exec;
     using REDUCE_POLICY = RAJA::omp_reduce;
     #else
-    using EXEC_POLICY = RAJA::loop_exec;
+    using EXEC_POLICY = RAJA::seq_exec;
     using REDUCE_POLICY = RAJA::seq_reduce;
     #endif
 
diff --git a/unitTests/testUtils.hpp b/unitTests/testUtils.hpp
@@ -36,7 +36,7 @@ template< typename >
 struct RAJAHelper
 {};
 
-using serialPolicy = RAJA::loop_exec;
+using serialPolicy = RAJA::seq_exec;
 
 template<>
 struct RAJAHelper< serialPolicy >

Original file line number	Diff line number	Diff line change
`@@ -350,7 +350,7 @@ CUDA_TEST( Array, chaiBuffer )`
`350`	`350`	`LvArray::ChaiBuffer > const & view = array;`
`351`	`351`
`352`	`352`	`// Capture the view in a host kernel which moves the data back to the host.`
`353`		`- RAJA::forall< RAJA::loop_exec >(`
	`353`	`+ RAJA::forall< RAJA::seq_exec >(`
`354`	`354`	`RAJA::TypedRangeSegment< std::ptrdiff_t >( 0, view.size() ),`
`355`	`355`	`[view] ( std::ptrdiff_t const i )`
`356`	`356`	`{`