Table::copy: add assertion (#5350)

WeiqunZhang · web-flow · commit b1bd66b8a518 · 2026-04-21T09:29:10.000-07:00
diff --git a/Src/Base/AMReX_TableData.H b/Src/Base/AMReX_TableData.H
@@ -627,6 +627,8 @@ void
 TableData<T,N,ORDER>::copy (TableData<T,N,ORDER> const& rhs) noexcept
 {
     std::size_t count = sizeof(T)*size();
+    if (count == 0) { return; }
+    AMREX_ASSERT(count == sizeof(T)*rhs.size());
 #ifdef AMREX_USE_GPU
     bool this_on_device = arena()->isManaged() || arena()->isDevice();
     bool rhs_on_device = rhs.arena()->isManaged() || rhs.arena()->isDevice();

Original file line number	Diff line number	Diff line change
`@@ -627,6 +627,8 @@ void`
`627`	`627`	`TableData<T,N,ORDER>::copy (TableData<T,N,ORDER> const& rhs) noexcept`
`628`	`628`	`{`
`629`	`629`	`std::size_t count = sizeof(T)*size();`
	`630`	`+ if (count == 0) { return; }`
	`631`	`+ AMREX_ASSERT(count == sizeof(T)*rhs.size());`
`630`	`632`	`#ifdef AMREX_USE_GPU`
`631`	`633`	`bool this_on_device = arena()->isManaged() \|\| arena()->isDevice();`
`632`	`634`	`bool rhs_on_device = rhs.arena()->isManaged() \|\| rhs.arena()->isDevice();`