Add a VecGeom+RelWithDebInfo+debug build to the CI

.jenkins

@@ -86,6 +86,22 @@ pipeline {
             }
           }
         }
+        stage('vecgeom-reldeb') {
+          agent {
+            docker {
+              image 'celeritas/ci-jammy-cuda11:2023-03-13'
+              label 'NVIDIA_Tesla_V100-PCIE-32GB && nvidia-docker && large_images'
+            }
+          }
+          steps {
+            sh 'entrypoint-shell ./scripts/ci/run-ci.sh ubuntu-cuda vecgeom-reldeb'
+          }
+          post {
+            always {
+              xunit reduceLog: false, tools:[CTest(deleteOutputFiles: true, failIfNotNew: true, pattern: 'build/Testing/**/Test.xml', skipNoTestFiles: false, stopProcessingIfError: true)]
+            }
+          }
+        }
         stage('vecgeom-demos') {
           agent {
             docker {

scripts/cmake-presets/ci-ubuntu-cuda.json

@@ -59,6 +59,11 @@
       "description": "Build everything but VecGeom in release mode",
       "inherits": [".ndebug", "full-novg"]
     },
+    {
+      "name": "vecgeom-reldeb",
+      "description": "Build with RelWithDebInfo, assertions, and VecGeom",
+      "inherits": [".reldeb", ".vecgeom", "base"]
+    },
     {
       "name": "vecgeom-tests",
       "description": "Build tests in debug with vecgeom",
@@ -88,6 +93,7 @@
     },
     {"name": "full-novg"       , "configurePreset": "full-novg"       , "inherits": "base"},
     {"name": "full-novg-ndebug", "configurePreset": "full-novg-ndebug", "inherits": "base", "targets": ["all", "install"]},
+    {"name": "vecgeom-reldeb"  , "configurePreset": "vecgeom-reldeb"  , "inherits": "base", "targets": ["all", "install"]},
     {"name": "vecgeom-tests"   , "configurePreset": "vecgeom-tests"   , "inherits": "base", "jobs": 8},
     {"name": "vecgeom-demos"   , "configurePreset": "vecgeom-demos"   , "inherits": "base", "jobs": 8, "targets": ["app/all", "install"]}
   ],
@@ -109,6 +115,7 @@
     },
     {"name": "full-novg"       , "configurePreset": "full-novg"       , "inherits": "base"},
     {"name": "full-novg-ndebug", "configurePreset": "full-novg-ndebug", "inherits": "base"},
+    {"name": "vecgeom-reldeb"  , "configurePreset": "vecgeom-reldeb"  , "inherits": "base"},
     {"name": "vecgeom-tests"   , "configurePreset": "vecgeom-tests"   , "inherits": "base"},
     {"name": "vecgeom-demos"   , "configurePreset": "vecgeom-demos"   , "inherits": "base",
       "filter": {

src/accel/detail/HitProcessor.cc

@@ -294,7 +294,7 @@ bool HitProcessor::update_touchable(Real3 const& pos,
     // Check that physical and logical volumes are consistent
     G4VPhysicalVolume* pv = touchable->GetVolume(0);
     CELER_ASSERT(pv);
-    if (!CELER_UNLIKELY(pv->GetLogicalVolume() != lv))
+    if (!CELER_UNLIKELY((pv->GetLogicalVolume() != lv)))
     {
         return true;
     }

src/celeritas/ext/detail/GenericPlacedVolume.hh

@@ -231,10 +231,7 @@ class GenericPlacedVolume : public vecgeom::VPlacedVolume
     }
 
 #ifdef VECGEOM_ROOT
-    virtual TGeoShape const* ConvertToRoot() const override
-    {
-        return nullptr;
-    }
+    virtual TGeoShape const* ConvertToRoot() const override { return nullptr; }
 #endif
 
     //! Dispatch to VecGeom function for shape's bounding box in its local
@@ -272,10 +269,7 @@ class GenericPlacedVolume : public vecgeom::VPlacedVolume
     //
     // These implementations are required when CUDA is enabled.
     // A trivialimplementation is okay, since won't be called from GPU.
-    virtual size_t DeviceSizeOf() const override
-    {
-        return 0;
-    }
+    virtual size_t DeviceSizeOf() const override { return 0; }
 
     DevicePtr<CudaVPlacedVolume>
     CopyToGpu(DevicePtr<CudaLogicalVolume> const,

src/celeritas/ext/detail/GenericSolid.hh

@@ -213,10 +213,7 @@ class GenericSolid : public GenericSolidBase
     //
     // These implementations are required when CUDA is enabled.
     // A trivialimplementation is okay, since won't be called from GPU.
-    size_t DeviceSizeOf() const
-    {
-        return 0;
-    }
+    size_t DeviceSizeOf() const { return 0; }
     DevicePtr<CudaUnplacedVolume> CopyToGpu() const
     {
         CELER_NOT_IMPLEMENTED("GenericSolid with CUDA");

src/celeritas/user/SimpleCaloData.cc

@@ -19,9 +19,9 @@ namespace celeritas
  */
 template<MemSpace M>
 void resize(SimpleCaloStateData<Ownership::value, M>* state,
-                   HostCRef<SimpleCaloParamsData> const& params,
-                   StreamId,
-                   size_type num_track_slots)
+            HostCRef<SimpleCaloParamsData> const& params,
+            StreamId,
+            size_type num_track_slots)
 {
     CELER_EXPECT(params);
     resize(&state->energy_deposition, params.num_detectors);

src/corecel/data/CollectionAlgorithms.hh

@@ -20,7 +20,8 @@ namespace celeritas
 template<class T, Ownership W, MemSpace M, class I>
 void fill(T&& value, Collection<T, W, M, I>* col)
 {
-    static_assert(W != Ownership::const_reference, "const references cannot be filled");
+    static_assert(W != Ownership::const_reference,
+                  "const references cannot be filled");
     CELER_EXPECT(col);
     detail::Filler<T, M> fill_impl{value};
     fill_impl((*col)[AllItems<T, M>{}]);