promise: update comments/names to reflect usage
authorBrian Norris <banorris@uci.edu>
Thu, 7 Feb 2013 00:51:12 +0000 (16:51 -0800)
committerBrian Norris <banorris@uci.edu>
Thu, 7 Feb 2013 00:52:30 +0000 (16:52 -0800)
is_compatible() and is_compatible_exclusive() do not require a store as
an argument; they can just as well use a load

promise.cc
promise.h

index ea5e7e635cf47df48a2ffc15fa0cd35d73b76025..69197cbac818b8f8dc01ad75d1d109df61474783 100644 (file)
@@ -78,21 +78,23 @@ bool Promise::has_failed() const
 }
 
 /**
- * @param write A store which could satisfy this Promise
- * @return True if the store can satisfy this Promise; false otherwise
+ * @brief Check if an action's thread and location are compatible for resolving
+ * this promise
+ * @param act The action to check against
+ * @return True if we are compatible; false otherwise
  */
-bool Promise::is_compatible(const ModelAction *write) const
+bool Promise::is_compatible(const ModelAction *act) const
 {
-       return thread_is_available(write->get_tid()) && read->same_var(write);
+       return thread_is_available(act->get_tid()) && read->same_var(act);
 }
 
 /**
- * @brief Check if a promise is compatible with a store and is exclusive to its
- * thread
- * @param write The store to check against
+ * @brief Check if an action's thread and location are compatible for resolving
+ * this promise, and that the promise is thread-exclusive
+ * @param act The action to check against
  * @return True if we are compatible and exclusive; false otherwise
  */
-bool Promise::is_compatible_exclusive(const ModelAction *write) const
+bool Promise::is_compatible_exclusive(const ModelAction *act) const
 {
-       return get_num_available_threads() == 1 && is_compatible(write);
+       return get_num_available_threads() == 1 && is_compatible(act);
 }
index 178b86e7d4b3cedba32f2240f2493e63655efb4d..852fe714f896dfbffc0b409fcd499058841a1ee0 100644 (file)
--- a/promise.h
+++ b/promise.h
@@ -41,8 +41,8 @@ class Promise {
        void set_write(const ModelAction *act) { write = act; }
        const ModelAction * get_write() const { return write; }
        int get_num_available_threads() const { return num_available_threads; }
-       bool is_compatible(const ModelAction *write) const;
-       bool is_compatible_exclusive(const ModelAction *write) const;
+       bool is_compatible(const ModelAction *act) const;
+       bool is_compatible_exclusive(const ModelAction *act) const;
 
        void print() const;