From eace8edd98fe684e4631d9afb9be76ad3081f746 Mon Sep 17 00:00:00 2001 From: Boris Kolpackov Date: Fri, 28 Jul 2017 09:54:39 +0200 Subject: Fix bug in target_count logic --- build2/target.ixx | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) (limited to 'build2/target.ixx') diff --git a/build2/target.ixx b/build2/target.ixx index 48036be..459265b 100644 --- a/build2/target.ixx +++ b/build2/target.ixx @@ -147,11 +147,18 @@ namespace build2 if (f != nullptr && *f == &noop_action) state_ = target_state::unchanged; else + { // This gets tricky when we start considering overrides (which can // only happen for noop recipes), direct execution, etc. So here seems // like the best place to do this. // - target_count.fetch_add (1, memory_order_relaxed); + // We also ignore the group recipe since it is used for ad hoc + // groups (which are not executed). Plus, group action means real + // recipe is in the group so this also feels right conceptually. + // + if (f == nullptr || *f != &group_action) + target_count.fetch_add (1, memory_order_relaxed); + } } } -- cgit v1.1