From d1d1d444f0b4abbb628625240025bfe3cfe77459 Mon Sep 17 00:00:00 2001
From: Boris Kolpackov <boris@codesynthesis.com>
Date: Mon, 6 Mar 2017 09:57:15 +0200
Subject: Tweak target locking logic

---
 build2/target.cxx | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

(limited to 'build2/target.cxx')

diff --git a/build2/target.cxx b/build2/target.cxx
index dacf534..fa8378c 100644
--- a/build2/target.cxx
+++ b/build2/target.cxx
@@ -149,15 +149,12 @@ namespace build2
       if (e >= busy)
         return target_state::unchanged; // Override in progress.
 
+      // Unlike lock_impl(), we are only called after being matched for this
+      // action so if we see executed, then it means executed for this action
+      // (or noop).
+      //
       if (e == exec)
-      {
-        // Sanity check: we better not be overriding a recipe for an already
-        // executed target.
-        //
-        assert (action == a);
-
         return group_state () ? group->state_ : state_;
-      }
 
       // Try to grab the spin-lock.
       //
-- 
cgit v1.1