diff options
author | Boris Kolpackov <boris@codesynthesis.com> | 2022-04-15 10:41:40 +0200 |
---|---|---|
committer | Boris Kolpackov <boris@codesynthesis.com> | 2022-04-15 10:41:40 +0200 |
commit | 642d2f1d3f83e3f5e303cf75b9a9d0a563e4ada4 (patch) | |
tree | fdb34bb27c6ff9de456ee899529ccad865423340 /libbuild2/context.cxx | |
parent | 65fadc6003259719161be650d2ede16c30e0bb3b (diff) |
Omit unnecessary clearing of cached base_scope values
Diffstat (limited to 'libbuild2/context.cxx')
-rw-r--r-- | libbuild2/context.cxx | 6 |
1 files changed, 6 insertions, 0 deletions
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 " |