Don't copy LinuxPerf Stats

It's not clear to me how deep the copying should be, but I think this should be fine for now and it matches the semantics of `copy` better.
JuliaCI · Jan 8, 2024 · 6503496 · 6503496
1 parent bbfb733
commit 6503496
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 8 deletions.
diff --git a/src/parameters.jl b/src/parameters.jl
@@ -110,7 +110,7 @@ function Base.copy(p::Parameters)
         p.time_tolerance,
         p.memory_tolerance,
         p.enable_linux_perf,
-        copy(p.linux_perf_options),
+        p.linux_perf_options,
     )
 end
 

diff --git a/src/trials.jl b/src/trials.jl
@@ -30,7 +30,7 @@ function Base.copy(t::Trial)
         copy(t.gctimes),
         t.memory,
         t.allocs,
-        isnothing(t.linux_perf_stats) ? nothing : copy(t.linux_perf_stats),
+        t.linux_perf_stats,
     )
 end
 
@@ -132,12 +132,7 @@ end
 
 function Base.copy(t::TrialEstimate)
     return TrialEstimate(
-        copy(t.params),
-        t.time,
-        t.gctime,
-        t.memory,
-        t.allocs,
-        isnothing(t.linux_perf_stats) ? nothing : copy(t.linux_perf_stats),
+        copy(t.params), t.time, t.gctime, t.memory, t.allocs, t.linux_perf_stats
     )
 end