ensure that only one eval is used for the warmup, test deprecations

JuliaCI · Jan 4, 2024 · f796677 · f796677
1 parent 8665689
commit f796677
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/execution.jl b/src/execution.jl
@@ -112,7 +112,7 @@ function _run(b::Benchmark, p::Parameters; verbose=false, pad="", kwargs...)
     params.gctrial && gcscrub()
     start_time = Base.time()
     trial = Trial(params)
-    b.samplefunc(b.quote_vals, params) #warmup sample
+    b.samplefunc(b.quote_vals, Parameters(params, evals=1)) #warmup sample
     params.gcsample && gcscrub()
     s = b.samplefunc(b.quote_vals, params)
     push!(trial, s[1:(end - 1)]...)
@@ -181,6 +181,7 @@ function _lineartrial(b::Benchmark, p::Parameters=b.params; maxevals=RESOLUTION,
     params = Parameters(p; kwargs...)
     estimates = zeros(maxevals)
     completed = 0
+    params.evals = 1
     b.samplefunc(b.quote_vals, params) #warmup sample
     params.gctrial && gcscrub()
     start_time = time()

diff --git a/test/ExecutionTests.jl b/test/ExecutionTests.jl
@@ -148,6 +148,8 @@ tune!(b_pass)
 # warmup #
 ###########
 
+@test_deprecated warmup(@benchmarkable sin(1))
+
 is_warm = false
 function needs_warm()
     global is_warm