From f9db4de0f3758e0f730a5d98348e7cc40082104a Mon Sep 17 00:00:00 2001
From: pennae <github@quasiparticle.net>
Date: Mon, 11 Dec 2023 15:54:16 +0100
Subject: [PATCH] force-inline forceValue
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

forceValue is extremely hot. interestingly adding likeliness annotations
to the branches does not seem to make a difference.

before:

  Time (mean ± σ):      4.224 s ±  0.005 s    [User: 3.711 s, System: 0.512 s]
  Range (min … max):    4.218 s …  4.234 s    10 runs

after:

  Time (mean ± σ):      4.140 s ±  0.009 s    [User: 3.647 s, System: 0.492 s]
  Range (min … max):    4.130 s …  4.152 s    10 runs
---
 src/libexpr/eval-inline.hh | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/libexpr/eval-inline.hh b/src/libexpr/eval-inline.hh
index 8a9ebb77a..d48871628 100644
--- a/src/libexpr/eval-inline.hh
+++ b/src/libexpr/eval-inline.hh
@@ -81,6 +81,7 @@ Env & EvalState::allocEnv(size_t size)
 }
 
 
+[[gnu::always_inline]]
 void EvalState::forceValue(Value & v, const PosIdx pos)
 {
     if (v.isThunk()) {