about summary refs log tree commit diff
diff options
context:
space:
mode:
authorEelco Dolstra <eelco.dolstra@logicblox.com>2015-07-23T21·11+0200
committerEelco Dolstra <eelco.dolstra@logicblox.com>2015-07-23T21·11+0200
commitc8bb2371ebf25ab78abde84e3c5742f9844a74b7 (patch)
tree53aeea8ded7f7b21636a87414f715f897962ffcd
parent16c9935fa9487f504b1e9c7f14d0f85ac870a62c (diff)
Optimize empty sets
This reduces the number of Bindings allocations by about 10%.
-rw-r--r--src/libexpr/attr-set.cc10
-rw-r--r--src/libexpr/eval.cc4
-rw-r--r--src/libexpr/eval.hh4
3 files changed, 14 insertions, 4 deletions
diff --git a/src/libexpr/attr-set.cc b/src/libexpr/attr-set.cc
index 9ac5e603199a..910428c02686 100644
--- a/src/libexpr/attr-set.cc
+++ b/src/libexpr/attr-set.cc
@@ -29,13 +29,17 @@ Bindings * EvalState::allocBindings(Bindings::size_t capacity)
 }
 
 
-void EvalState::mkAttrs(Value & v, unsigned int expected)
+void EvalState::mkAttrs(Value & v, unsigned int capacity)
 {
+    if (capacity == 0) {
+        v = vEmptySet;
+        return;
+    }
     clearValue(v);
     v.type = tAttrs;
-    v.attrs = allocBindings(expected);
+    v.attrs = allocBindings(capacity);
     nrAttrsets++;
-    nrAttrsInAttrsets += expected;
+    nrAttrsInAttrsets += capacity;
 }
 
 
diff --git a/src/libexpr/eval.cc b/src/libexpr/eval.cc
index 044256112d50..14169d85735e 100644
--- a/src/libexpr/eval.cc
+++ b/src/libexpr/eval.cc
@@ -281,6 +281,10 @@ EvalState::EvalState(const Strings & _searchPath)
     for (auto & i : paths) addToSearchPath(i);
     addToSearchPath("nix=" + settings.nixDataDir + "/nix/corepkgs");
 
+    clearValue(vEmptySet);
+    vEmptySet.type = tAttrs;
+    vEmptySet.attrs = allocBindings(0);
+
     createBaseEnv();
 }
 
diff --git a/src/libexpr/eval.hh b/src/libexpr/eval.hh
index 1b546f89c836..80eba975e9c3 100644
--- a/src/libexpr/eval.hh
+++ b/src/libexpr/eval.hh
@@ -80,6 +80,8 @@ public:
        path or to environment variables. */
     bool restricted;
 
+    Value vEmptySet;
+
 private:
     SrcToStore srcToStore;
 
@@ -227,7 +229,7 @@ public:
     Bindings * allocBindings(Bindings::size_t capacity);
 
     void mkList(Value & v, unsigned int length);
-    void mkAttrs(Value & v, unsigned int expected);
+    void mkAttrs(Value & v, unsigned int capacity);
     void mkThunk_(Value & v, Expr * expr);
     void mkPos(Value & v, Pos * pos);