Undo accidental push to master

src/libexpr/eval.cc

@@ -232,15 +232,6 @@ EvalMemory::EvalMemory()
     assertGCInitialized();
 }
 
-Value * EvalMemory::allocInt(NixInt::Inner n)
-{
-    if (n >= 0 && n < (NixInt::Inner) Value::vSmallInts.size())
-        return &Value::vSmallInts[n];
-    Value * v = allocValue();
-    v->mkInt(NixInt(n));
-    return v;
-}
-
 EvalState::EvalState(
     const LookupPath & lookupPathFromArguments,
     ref<Store> store,

src/libexpr/include/nix/expr/eval.hh

@@ -357,8 +357,6 @@ public:
         return stats;
     }
 
-    Value * allocInt(NixInt::Inner n);
-
     /**
      * Storage for the AST nodes
      */

src/libexpr/include/nix/expr/value.hh

@@ -1016,13 +1016,6 @@ struct Value : public ValueStorage<sizeof(void *)>
      */
     static Value vFalse;
 
-    /**
-     * Small pre-allocated integer constants.
-     *
-     * These are _not_ singletons. Pointer equality is _not_ sufficient.
-     */
-    static std::array<Value, 32> vSmallInts;
-
 private:
     template<InternalType... discriminator>
     bool isa() const noexcept

src/libexpr/primops.cc

@@ -3161,7 +3161,8 @@ static struct LazyPosAccessors
 
     void operator()(EvalState & state, const PosIdx pos, Value & line, Value & column)
     {
-        auto posV = state.mem.allocInt(pos.id);
+        Value * posV = state.allocValue();
+        posV->mkInt(pos.id);
         line.mkApp(&lineOfPos, posV);
         column.mkApp(&columnOfPos, posV);
     }
@@ -3965,8 +3966,11 @@ static void prim_genList(EvalState & state, const PosIdx pos, Value ** args, Val
     state.forceFunction(*args[0], noPos, "while evaluating the first argument passed to builtins.genList");
 
     auto list = state.buildList(len);
-    for (const auto & [n, v] : enumerate(list))
-        (v = state.allocValue())->mkApp(args[0], state.mem.allocInt(n));
+    for (const auto & [n, v] : enumerate(list)) {
+        auto arg = state.allocValue();
+        arg->mkInt(n);
+        (v = state.allocValue())->mkApp(args[0], arg);
+    }
     v.mkList(list);
 }
 

src/libexpr/primops/meson.build

@@ -9,5 +9,4 @@ sources += files(
   'fetchMercurial.cc',
   'fetchTree.cc',
   'fromTOML.cc',
-  'imap.cc',
 )

src/libexpr/value.cc

@@ -26,11 +26,4 @@ Value Value::vFalse = []() {
     return res;
 }();
 
-std::array<Value, 32> Value::vSmallInts = []() {
-    decltype(Value::vSmallInts) arr;
-    for (size_t i = 0; i < arr.size(); ++i)
-        arr[i].mkInt(i);
-    return arr;
-}();
-
 } // namespace nix