From: Brian Norris <banorris@uci.edu>
Date: Mon, 20 Aug 2012 22:59:25 +0000 (-0700)
Subject: action: update 'reads_from' field before synchronization
X-Git-Tag: pldi2013~256
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=9115c8a01b0367665b4b7b3f74dc63a375ac81c6;p=model-checker.git

action: update 'reads_from' field before synchronization

As the ModelChecker's release/acquire functionality gets more complex,
it makes more sense to assign the ModelAction::reads_from field before
checking synchronization. Currently, this makes no change in behavior.
---

diff --git a/action.cc b/action.cc
index 5c036ef..b9e9c02 100644
--- a/action.cc
+++ b/action.cc
@@ -165,10 +165,10 @@ void ModelAction::create_cv(const ModelAction *parent)
 void ModelAction::read_from(const ModelAction *act)
 {
 	ASSERT(cv);
+	reads_from = act;
 	if (act!=NULL && act->is_release() && this->is_acquire()) {
 		synchronize_with(act);
 	}
-	reads_from = act;
 }
 
 /**