From: Brian Norris <banorris@uci.edu>
Date: Wed, 3 Oct 2012 22:36:26 +0000 (-0700)
Subject: model: do not assume THREAD_FINISH is always the last action
X-Git-Tag: pldi2013~107^2~4
X-Git-Url: http://plrg.eecs.uci.edu/git/?p=model-checker.git;a=commitdiff_plain;h=cf4cc8a444d3c85367b9aa15b91b2829220e2edf;ds=sidebyside

model: do not assume THREAD_FINISH is always the last action

It's possible that there will be special fixup ModelActions added, so
just use the Thread::is_complete() method to check for a completed
Thread.
---

diff --git a/model.cc b/model.cc
index 788a670..326358e 100644
--- a/model.cc
+++ b/model.cc
@@ -1246,7 +1246,7 @@ bool ModelChecker::release_seq_head(const ModelAction *rf, rel_heads_list_t *rel
 
 		ModelAction *last = get_last_action(int_to_id(i));
 		if (last && (rf->happens_before(last) ||
-				last->get_type() == THREAD_FINISH))
+				get_thread(int_to_id(i))->is_complete()))
 			future_ordered = true;
 
 		for (rit = list->rbegin(); rit != list->rend(); rit++) {