From 9034f7c51ef6437ce9d4547ba5bde217b4740fb2 Mon Sep 17 00:00:00 2001
From: Boris Kolpackov <boris@codesynthesis.com>
Date: Tue, 19 Apr 2022 11:10:53 +0200
Subject: Use target recipe for auxiliary data storage during match-apply

In particular, we now have separate auxiliary data storage for inner
and outer operations.
---
 libbuild2/adhoc-rule-buildscript.cxx | 22 +++++++++++++++++++---
 1 file changed, 19 insertions(+), 3 deletions(-)

(limited to 'libbuild2/adhoc-rule-buildscript.cxx')

diff --git a/libbuild2/adhoc-rule-buildscript.cxx b/libbuild2/adhoc-rule-buildscript.cxx
index aa30552..03e810d 100644
--- a/libbuild2/adhoc-rule-buildscript.cxx
+++ b/libbuild2/adhoc-rule-buildscript.cxx
@@ -470,10 +470,22 @@ namespace build2
     //
     if (a != perform_update_id || !xt.is_a<file> ())
     {
-      return [d, this] (action a, const target& t)
+      // Make sure we get small object optimization.
+      //
+      if (d)
       {
-        return default_action (a, t, d);
-      };
+        return [dv = *d, this] (action a, const target& t)
+        {
+          return default_action (a, t, dv);
+        };
+      }
+      else
+      {
+        return [this] (action a, const target& t)
+        {
+          return default_action (a, t, nullopt);
+        };
+      }
     }
 
     // See if this is the simple case with only static dependencies.
@@ -589,6 +601,10 @@ namespace build2
       }
     }
 
+    // Note that while it's tempting to turn match_data* into recipes, some of
+    // their members are not movable. And in the end we will have the same
+    // result: one dynamic memory allocation.
+    //
     unique_ptr<match_data> md;
     unique_ptr<match_data_byproduct> mdb;
 
-- 
cgit v1.1