change the way to detect loops
[c11tester.git] / funcnode.cc
index 7c14655065f018e5b990567b2b363d10cb9e1194..085ab5bd49e2d2e12ebc82402bf42a39da727905 100644 (file)
@@ -1,15 +1,16 @@
 #include "funcnode.h"
-#include <fcntl.h>
 
 FuncNode::FuncNode() :
        predicate_tree_initialized(false),
+       predicate_tree_entry(new Predicate(NULL, true)),
+       exit_count(0),
        func_inst_map(),
        inst_list(),
        entry_insts(),
-       thrd_read_map(),
-       predicate_tree_entry(),
-       inst_pred_map()
-{}
+       thrd_read_map()
+{
+       predicate_tree_entry->add_predicate_expr(NOPREDICATE, NULL, true);
+}
 
 /* Check whether FuncInst with the same type, position, and location
  * as act has been added to func_inst_map or not. If not, add it.
@@ -31,17 +32,9 @@ void FuncNode::add_inst(ModelAction *act)
        if ( func_inst_map.contains(position) ) {
                FuncInst * inst = func_inst_map.get(position);
 
-               if (inst->get_type() != act->get_type() ) {
-                       // model_print("action with a different type occurs at line number %s\n", position);
-                       FuncInst * func_inst = inst->search_in_collision(act);
-
-                       if (func_inst != NULL)
-                               return;
-
-                       func_inst = new FuncInst(act, this);
-                       inst->get_collisions()->push_back(func_inst);
-                       inst_list.push_back(func_inst); // delete?
-               }
+               ASSERT(inst->get_type() == act->get_type());
+               if (inst->get_location() != act->get_location())
+                       inst->not_single_location();
 
                return;
        }
@@ -112,8 +105,7 @@ void FuncNode::update_tree(action_list_t * act_list)
 
        /* build inst_list from act_list for later processing */
        func_inst_list_t inst_list;
-       func_inst_list_t read_inst_list;
-       HashTable<FuncInst *, uint64_t, uintptr_t, 4> read_val_map;
+       action_list_t read_act_list;
 
        for (sllnode<ModelAction *> * it = act_list->begin(); it != NULL; it = it->getNext()) {
                ModelAction * act = it->getVal();
@@ -124,17 +116,16 @@ void FuncNode::update_tree(action_list_t * act_list)
 
                inst_list.push_back(func_inst);
 
-//             model_print("position: %s ", act->get_position());
-//             act->print();
-
-               if (func_inst->is_read()) {
-                       read_inst_list.push_back(func_inst);
-                       read_val_map.put(func_inst, act->get_reads_from_value());
-               }
+               if (func_inst->is_read())
+                       read_act_list.push_back(act);
        }
 
+//     model_print("function %s\n", func_name);
        update_inst_tree(&inst_list);
-       update_predicate_tree(&read_inst_list, &read_val_map);
+       update_predicate_tree(&read_act_list);
+//     deep_update(predicate_tree_entry);
+
+       print_predicate_tree();
 }
 
 /** 
@@ -197,13 +188,13 @@ void FuncNode::store_read(ModelAction * act, uint32_t tid)
 uint64_t FuncNode::query_last_read(void * location, uint32_t tid)
 {
        if (thrd_read_map.size() <= tid)
-               return 0xdeadbeef;
+               return VALUE_NONE;
 
        read_map_t * read_map = thrd_read_map[tid];
 
        /* last read value not found */
        if ( !read_map->contains(location) )
-               return 0xdeadbeef;
+               return VALUE_NONE;
 
        uint64_t read_val = read_map->get(location);
        return read_val;
@@ -221,9 +212,9 @@ void FuncNode::clear_read_map(uint32_t tid)
        thrd_read_map[tid]->reset();
 }
 
-void FuncNode::update_predicate_tree(func_inst_list_t * inst_list, HashTable<FuncInst *, uint64_t, uintptr_t, 4> * read_val_map)
+void FuncNode::update_predicate_tree(action_list_t * act_list)
 {
-       if (inst_list == NULL || inst_list->size() == 0)
+       if (act_list == NULL || act_list->size() == 0)
                return;
 /*
        if (predicate_tree_initialized) {
@@ -231,74 +222,156 @@ void FuncNode::update_predicate_tree(func_inst_list_t * inst_list, HashTable<Fun
        }
        predicate_tree_initialized = true;
 */
-       // maybe restrict the size of hashtable to save calloc time
-       HashTable<void *, FuncInst *, uintptr_t, 4> loc_inst_map(64);
+       /* map a FuncInst to the its predicate */
+       HashTable<FuncInst *, Predicate *, uintptr_t, 0> inst_pred_map(128);
 
-       sllnode<FuncInst *> *it = inst_list->begin();
-       FuncInst * entry_inst = it->getVal();
+       // number FuncInsts to detect loops
+       HashTable<FuncInst *, uint32_t, uintptr_t, 0> inst_id_map(128);
+       uint32_t inst_counter = 0;
 
-       /* get the unique Predicate pointer, assuming entry instructions have no predicate expression */
-       Predicate * curr_pred = NULL;
-       PredSetIter * pit = predicate_tree_entry.iterator();
-       while (pit->hasNext()) {
-               Predicate * p = pit->next();
-               if (p->get_func_inst() == entry_inst) {
-                       curr_pred = p;
-                       break;
+       HashTable<void *, ModelAction *, uintptr_t, 0> loc_act_map(128);
+       HashTable<FuncInst *, ModelAction *, uintptr_t, 0> inst_act_map(128);
+
+       sllnode<ModelAction *> *it = act_list->begin();
+       Predicate * curr_pred = predicate_tree_entry;
+       while (it != NULL) {
+               ModelAction * next_act = it->getVal();
+               FuncInst * next_inst = get_inst(next_act);
+               SnapVector<Predicate *> * unset_predicates = new SnapVector<Predicate *>();
+
+               bool branch_found = follow_branch(&curr_pred, next_inst, next_act, &inst_act_map, unset_predicates);
+
+               // no predicate expressions, follow the only branch
+               if (!branch_found && unset_predicates->size() != 0) {
+                       ASSERT(unset_predicates->size() == 1);
+                       Predicate * one_branch = (*unset_predicates)[0];
+                       curr_pred = one_branch;
+                       branch_found = true;
                }
-       }
-       if (curr_pred == NULL) {
-               curr_pred = new Predicate(entry_inst);
-               curr_pred->set_entry_predicate();
-               predicate_tree_entry.add(curr_pred);
-       }
 
-       loc_inst_map.put(entry_inst->get_location(), entry_inst);
+               delete unset_predicates;
 
-       it = it->getNext();
-       while (it != NULL) {
-               FuncInst * curr_inst = it->getVal();
-               bool branch_found = follow_branch(&curr_pred, curr_inst, read_val_map, &loc_inst_map);
+               // detect loops
+               if (!branch_found && inst_id_map.contains(next_inst)) {
+                       FuncInst * curr_inst = curr_pred->get_func_inst();
+                       uint32_t curr_id = inst_id_map.get(curr_inst);
+                       uint32_t next_id = inst_id_map.get(next_inst);
+
+                       if (curr_id >= next_id) {
+                               Predicate * old_pred = inst_pred_map.get(next_inst);
+                               Predicate * back_pred = old_pred->get_parent();
+
+                               curr_pred->add_backedge(back_pred);
+                               curr_pred = back_pred;
+
+                               continue;
+                       }
+               }
 
                if (!branch_found) {
-                       if ( loc_inst_map.contains(curr_inst->get_location()) ) {
-                               Predicate * new_pred1 = new Predicate(curr_inst);
-                               new_pred1->add_predicate(EQUALITY, curr_inst->get_location(), true);
+                       if ( loc_act_map.contains(next_act->get_location()) ) {
+                               ModelAction * last_act = loc_act_map.get(next_act->get_location());
+                               FuncInst * last_inst = get_inst(last_act);
+
+                               Predicate * new_pred1 = new Predicate(next_inst);
+                               new_pred1->add_predicate_expr(EQUALITY, last_inst, true);
 
-                               Predicate * new_pred2 = new Predicate(curr_inst);
-                               new_pred2->add_predicate(EQUALITY, curr_inst->get_location(), false);
+                               Predicate * new_pred2 = new Predicate(next_inst);
+                               new_pred2->add_predicate_expr(EQUALITY, last_inst, false);
 
-                               /* TODO: add to inst_pred_map */
                                curr_pred->add_child(new_pred1);
                                curr_pred->add_child(new_pred2);
+                               new_pred1->set_parent(curr_pred);
+                               new_pred2->set_parent(curr_pred);
 
-                               FuncInst * last_inst = loc_inst_map.get(curr_inst->get_location());
-                               uint64_t last_read = read_val_map->get(last_inst);
-                               if ( last_read == read_val_map->get(curr_inst) )
+                               uint64_t last_read = last_act->get_reads_from_value();
+                               uint64_t next_read = next_act->get_reads_from_value();
+
+                               if ( last_read == next_read )
+                                       curr_pred = new_pred1;
+                               else
+                                       curr_pred = new_pred2;
+                       } else if (!next_inst->is_single_location()) {
+                               Predicate * new_pred1 = new Predicate(next_inst);
+                               new_pred1->add_predicate_expr(NULLITY, NULL, true);
+
+                               Predicate * new_pred2 = new Predicate(next_inst);
+                               new_pred2->add_predicate_expr(NULLITY, NULL, false);
+
+                               curr_pred->add_child(new_pred1);
+                               curr_pred->add_child(new_pred2);
+                               new_pred1->set_parent(curr_pred);
+                               new_pred2->set_parent(curr_pred);
+
+                               uint64_t next_read = next_act->get_reads_from_value();
+                               bool isnull = ((void*)next_read == NULL);
+                               if (isnull)
                                        curr_pred = new_pred1;
                                else
                                        curr_pred = new_pred2;
                        } else {
-                               Predicate * new_pred = new Predicate(curr_inst);
+                               Predicate * new_pred = new Predicate(next_inst);
                                curr_pred->add_child(new_pred);
+                               new_pred->set_parent(curr_pred);
+
+                               if (curr_pred->is_entry_predicate())
+                                       new_pred->add_predicate_expr(NOPREDICATE, NULL, true);
+
                                curr_pred = new_pred;
                        }
                }
 
-               loc_inst_map.put(curr_inst->get_location(), curr_inst);
+               inst_pred_map.put(next_inst, curr_pred);
+               if (!inst_id_map.contains(next_inst))
+                       inst_id_map.put(next_inst, inst_counter++);
 
+               loc_act_map.put(next_act->get_location(), next_act);
+               inst_act_map.put(next_inst, next_act);
                it = it->getNext();
        }
+}
 
-//     model_print("function %s\n", func_name);
-//     print_predicate_tree();
+void FuncNode::deep_update(Predicate * curr_pred)
+{
+       FuncInst * func_inst = curr_pred->get_func_inst();
+       if (func_inst != NULL && !func_inst->is_single_location()) {
+               bool has_null_pred = false;
+               PredExprSet * pred_expressions = curr_pred->get_pred_expressions();
+               PredExprSetIter * pred_expr_it = pred_expressions->iterator();
+               while (pred_expr_it->hasNext()) {
+                       pred_expr * pred_expression = pred_expr_it->next();
+                       if (pred_expression->token == NULLITY) {
+                               has_null_pred = true;
+                               break;
+                       }
+               }
+
+               if (!has_null_pred) {
+//                     func_inst->print();
+                       Predicate * another_branch = new Predicate(func_inst);
+                       another_branch->copy_predicate_expr(curr_pred);
+                       another_branch->add_predicate_expr(NULLITY, NULL, 1);
+                       curr_pred->add_predicate_expr(NULLITY, NULL, 0);
+
+                       Predicate * parent = curr_pred->get_parent();
+                       parent->add_child(another_branch);
+               }
+       }
+
+       ModelVector<Predicate *> * branches = curr_pred->get_children();
+       for (uint i = 0; i < branches->size(); i++) {
+               Predicate * branch = (*branches)[i];
+               deep_update(branch);
+       }
 }
 
-/* Given curr_pred and next_inst, find the branch following curr_pred that contains next_inst and the correct predicate
+/* Given curr_pred and next_inst, find the branch following curr_pred that
+ * contains next_inst and the correct predicate. 
  * @return true if branch found, false otherwise.
  */
-bool FuncNode::follow_branch(Predicate ** curr_pred, FuncInst * next_inst,
-       HashTable<FuncInst *, uint64_t, uintptr_t, 4> * read_val_map, HashTable<void *, FuncInst *, uintptr_t, 4> * loc_inst_map)
+bool FuncNode::follow_branch(Predicate ** curr_pred, FuncInst * next_inst, ModelAction * next_act,
+       HashTable<FuncInst *, ModelAction *, uintptr_t, 0> * inst_act_map,
+       SnapVector<Predicate *> * unset_predicates)
 {
        /* check if a branch with func_inst and corresponding predicate exists */
        bool branch_found = false;
@@ -308,43 +381,57 @@ bool FuncNode::follow_branch(Predicate ** curr_pred, FuncInst * next_inst,
                if (branch->get_func_inst() != next_inst)
                        continue;
 
+               /* check against predicate expressions */
+               bool predicate_correct = true;
                PredExprSet * pred_expressions = branch->get_pred_expressions();
+               PredExprSetIter * pred_expr_it = pred_expressions->iterator();
 
-               /* no predicate, follow the only branch */
                if (pred_expressions->getSize() == 0) {
-//                     model_print("no predicate exists: "); next_inst->print();
-                       *curr_pred = branch;
-                       branch_found = true;
-                       break;
+                       predicate_correct = false;
+                       unset_predicates->push_back(branch);
                }
 
-               PredExprSetIter * pred_expr_it = pred_expressions->iterator();
                while (pred_expr_it->hasNext()) {
                        pred_expr * pred_expression = pred_expr_it->next();
-                       uint64_t last_read, curr_read;
-                       FuncInst * last_inst;
+                       uint64_t last_read, next_read;
                        bool equality;
 
                        switch(pred_expression->token) {
+                               case NOPREDICATE:
+                                       predicate_correct = true;
+                                       break;
                                case EQUALITY:
-                                       last_inst = loc_inst_map->get(next_inst->get_location());
-                                       last_read = read_val_map->get(last_inst);
-                                       curr_read = read_val_map->get(next_inst);
-                                       equality = (last_read == curr_read);
-                                       if (equality == pred_expression->value) {
-                                               *curr_pred = branch;
-//                                             model_print("predicate: token: %d, location: %p, value: %d - ", pred_expression->token, pred_expression->location, pred_expression->value); next_inst->print();
-                                               branch_found = true;
-                                       }
+                                       FuncInst * to_be_compared;
+                                       ModelAction * last_act;
+
+                                       to_be_compared = pred_expression->func_inst;
+                                       last_act = inst_act_map->get(to_be_compared);
+
+                                       last_read = last_act->get_reads_from_value();
+                                       next_read = next_act->get_reads_from_value();
+                                       equality = (last_read == next_read);
+                                       if (equality != pred_expression->value)
+                                               predicate_correct = false;
+
                                        break;
                                case NULLITY:
+                                       next_read = next_act->get_reads_from_value();
+                                       equality = ((void*)next_read == NULL);
+                                       if (equality != pred_expression->value)
+                                               predicate_correct = false;
                                        break;
                                default:
+                                       predicate_correct = false;
                                        model_print("unkown predicate token\n");
                                        break;
                        }
                }
 
+               if (predicate_correct) {
+                       *curr_pred = branch;
+                       branch_found = true;
+                       break;
+               }
        }
 
        return branch_found;
@@ -353,12 +440,7 @@ bool FuncNode::follow_branch(Predicate ** curr_pred, FuncInst * next_inst,
 void FuncNode::print_predicate_tree()
 {
        model_print("digraph function_%s {\n", func_name);
-       PredSetIter * it = predicate_tree_entry.iterator();
-
-       while (it->hasNext()) {
-               Predicate * p = it->next();
-               p->print_pred_subtree();
-       }
+       predicate_tree_entry->print_pred_subtree();
        model_print("}\n");     // end of graph
 }