X-Git-Url: http://plrg.eecs.uci.edu/git/?p=c11tester.git;a=blobdiff_plain;f=clockvector.cc;h=54e8c4a1a9b5188eb0b56875015f13950712eabd;hp=4dd03dc26fb7291a59ed4d1304358948bff6e062;hb=89ecd60fab0d93d6df6aa35e663ab67db860fa1d;hpb=cabbdfb6fe631a0e4b1bfd1cb73862da6f04d7d8

diff --git a/clockvector.cc b/clockvector.cc
index 4dd03dc2..54e8c4a1 100644
--- a/clockvector.cc
+++ b/clockvector.cc
@@ -1,13 +1,13 @@
-#include <algorithm>
 #include <cstring>
 #include <stdlib.h>
 
-#include "model.h"
 #include "action.h"
+
 #include "clockvector.h"
 #include "common.h"
 #include "threads-model.h"
 
+
 /**
  * Constructs a new ClockVector, given a parent ClockVector and a first
  * ModelAction. This constructor can assign appropriate default settings if no
@@ -18,13 +18,16 @@
  */
 ClockVector::ClockVector(ClockVector *parent, ModelAction *act)
 {
-	num_threads = model->get_num_threads();
+	ASSERT(act);
+	num_threads = int_to_id(act->get_tid()) + 1;
+	if (parent && parent->num_threads > num_threads)
+		num_threads = parent->num_threads;
+
 	clock = (modelclock_t *)snapshot_calloc(num_threads, sizeof(int));
 	if (parent)
 		std::memcpy(clock, parent->clock, parent->num_threads * sizeof(modelclock_t));
 
-	if (act)
-		clock[id_to_int(act->get_tid())] = act->get_seq_number();
+	clock[id_to_int(act->get_tid())] = act->get_seq_number();
 }
 
 /** @brief Destructor */
@@ -38,21 +41,25 @@ ClockVector::~ClockVector()
  * resulting vector length will be the maximum length of the two being merged.
  * @param cv is the ClockVector being merged into this vector.
  */
-void ClockVector::merge(const ClockVector *cv)
+bool ClockVector::merge(const ClockVector *cv)
 {
 	ASSERT(cv != NULL);
-
+	bool changed = false;
 	if (cv->num_threads > num_threads) {
 		clock = (modelclock_t *)snapshot_realloc(clock, cv->num_threads * sizeof(modelclock_t));
-		for (int i= num_threads; i < cv->num_threads; i++)
+		for (int i = num_threads;i < cv->num_threads;i++)
 			clock[i] = 0;
 		num_threads = cv->num_threads;
 	}
 
 	/* Element-wise maximum */
-	for (int i = 0; i < cv->num_threads; i++)
-		if (cv->clock[i] > clock[i])
+	for (int i = 0;i < cv->num_threads;i++)
+		if (cv->clock[i] > clock[i]) {
 			clock[i] = cv->clock[i];
+			changed = true;
+		}
+
+	return changed;
 }
 
 /**
@@ -76,17 +83,6 @@ bool ClockVector::synchronized_since(const ModelAction *act) const
 	return false;
 }
 
-bool ClockVector::has_synchronized_with(const ClockVector *cv) const
-{
-	ASSERT(cv);
-	if (cv->num_threads > num_threads)
-		return false;
-	for (int i = 0; i < cv->num_threads; i++)
-		if (cv->clock[i] > clock[i])
-			return false;
-	return true;
-}
-
 /** Gets the clock corresponding to a given thread id from the clock vector. */
 modelclock_t ClockVector::getClock(thread_id_t thread) {
 	int threadid = id_to_int(thread);
@@ -101,7 +97,7 @@ modelclock_t ClockVector::getClock(thread_id_t thread) {
 void ClockVector::print() const
 {
 	int i;
-	printf("CV: (");
-	for (i = 0; i < num_threads; i++)
-		printf("%2u%s", clock[i], (i == num_threads - 1) ? ")\n" : ", ");
+	model_print("(");
+	for (i = 0;i < num_threads;i++)
+		model_print("%2u%s", clock[i], (i == num_threads - 1) ? ")\n" : ", ");
 }