From 922697c8b27570b6c76a472266507b73f6b79f7f Mon Sep 17 00:00:00 2001 From: Eelco Dolstra Date: Thu, 9 Mar 2006 15:09:18 +0000 Subject: * Big speedup (factor > 2.5) in all nix-env operations that do actual instantiation, e.g. "nix-env -i" and "nix-env -qas" (but not "nix-env -qa"). It turns out that many redundant calls to addToStore(path) were made, which reads and hashes the entire path. For instance, the bash bootstrap binary in Nixpkgs would be read and hashed many times. As a result nix-env would spend around 92% of its time in the function sha256_block (according to callgrind). Some simple memoization fixes this. --- src/libexpr/primops.cc | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) (limited to 'src/libexpr/primops.cc') diff --git a/src/libexpr/primops.cc b/src/libexpr/primops.cc index 31fff0b493e8..77721a5a8d87 100644 --- a/src/libexpr/primops.cc +++ b/src/libexpr/primops.cc @@ -165,9 +165,15 @@ static void processBinding(EvalState & state, Expr e, Derivation & drv, if (isDerivation(srcPath)) throw Error(format("file names are not allowed to end in `%1%'") % drvExtension); - Path dstPath(addToStore(srcPath)); - printMsg(lvlChatty, format("copied source `%1%' -> `%2%'") - % srcPath % dstPath); + Path dstPath; + if (state.srcToStore[srcPath] != "") + dstPath = state.srcToStore[srcPath]; + else { + dstPath = addToStore(srcPath); + state.srcToStore[srcPath] = dstPath; + printMsg(lvlChatty, format("copied source `%1%' -> `%2%'") + % srcPath % dstPath); + } drv.inputSrcs.insert(dstPath); ss.push_back(dstPath); } -- cgit 1.4.1