From 642d2f1d3f83e3f5e303cf75b9a9d0a563e4ada4 Mon Sep 17 00:00:00 2001
From: Boris Kolpackov <boris@codesynthesis.com>
Date: Fri, 15 Apr 2022 10:41:40 +0200
Subject: Omit unnecessary clearing of cached base_scope values

---
 libbuild2/context.cxx | 6 ++++++
 1 file changed, 6 insertions(+)

(limited to 'libbuild2/context.cxx')

diff --git a/libbuild2/context.cxx b/libbuild2/context.cxx
index 7944294..7b465b4 100644
--- a/libbuild2/context.cxx
+++ b/libbuild2/context.cxx
@@ -1072,11 +1072,17 @@ namespace build2
       //    number of files (targets) and a large number of load phase
       //    switches (due to directory buildfiles).
       //
+      // Thinking some more on this, we shouldn't need to do this since such
+      // loads can (or at least should) only perform "island appends" see
+      // comment on context::phase for details.
+      //
+#if 0
       if (*r)
       {
         for (const unique_ptr<target>& t: ctx.targets)
           t->base_scope_.store (nullptr, memory_order_relaxed);
       }
+#endif
     }
 
     //text << this_thread::get_id () << " phase switch  "
-- 
cgit v1.1