Merge pull request #276 from giordano/mg/btime-return

Make `@btime` correctly return the value of the expression in v1.8
JuliaCI · Feb 12, 2022 · cbf3786 · cbf3786 · vchuravy · Feb 12, 2022
2 parents bd02a60 + fae983b
commit cbf3786
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/Project.toml b/Project.toml
@@ -1,6 +1,6 @@
 name = "BenchmarkTools"
 uuid = "6e4b80f9-dd63-53aa-95a3-0cdb28fa8baf"
-version = "1.3"
+version = "1.3.1"
 
 [deps]
 JSON = "682c06a0-de6a-54ab-a142-c8b1cf79cde6"

diff --git a/src/execution.jl b/src/execution.jl
@@ -479,8 +479,11 @@ function generate_benchmark_definition(eval_module, out_vars, setup_vars, quote_
         invocation = :($(Expr(:tuple, out_vars...)) = $(signature))
         core_body = :($(core); $(returns))
     end
-    if isdefined(Base, :donotdelete)
-        invocation = :(Base.donotdelete($invocation))
+    @static if isdefined(Base, :donotdelete)
+        invocation = :(let x = $invocation
+                           Base.donotdelete(x)
+                           x
+                       end)
     end
     return Core.eval(eval_module, quote
         @noinline $(signature_def) = begin $(core_body) end