added missing initialization
[libfirm] / ir / tr / trvrfy.c
index 55ac610..6c707ea 100644 (file)
@@ -1,7 +1,21 @@
+/*
+ * Project:     libFIRM
+ * File name:   ir/tr/trvrfy.c
+ * Purpose:     Check types and entities for correctness.
+ * Author:      Michael Beck, Goetz Lindenmaier
+ * Modified by:
+ * Created:     29.1.2003
+ * CVS-ID:      $Id$
+ * Copyright:   (c) 2003 Universität Karlsruhe
+ * Licence:     This file protected by GPL -  GNU GENERAL PUBLIC LICENSE.
+ */
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
 
 #include "trvrfy.h"
 #include "irgraph_t.h"  /* for checking whether constant code is allocated
-                          on proper obstack */
+               on proper obstack */
 
 /**
  * Check a class
@@ -10,32 +24,35 @@ static int check_class(type *tp) {
   int i, j, k;
   int found;
 
-  //printf("\n"); DDMT(tp);
+  /*printf("\n"); DDMT(tp);*/
 
   for (i = 0; i < get_class_n_members(tp); i++) {
 
     entity *mem = get_class_member(tp, i);
     assert(mem && "NULL members not allowed");
-    //printf(" %d, %d", get_entity_n_overwrites(mem), get_class_n_supertypes(tp)); DDME(mem);
+    /*printf(" %d, %d", get_entity_n_overwrites(mem), get_class_n_supertypes(tp)); DDME(mem);*/
     if (!mem) return error_null_mem;
 
-    assert(get_entity_n_overwrites(mem) <= get_class_n_supertypes(tp));
+    if (get_entity_n_overwrites(mem) > get_class_n_supertypes(tp)) {
+      DDMT(tp); DDME(mem);
+      assert(get_entity_n_overwrites(mem) <= get_class_n_supertypes(tp));
+    }
     for (j = 0; j < get_entity_n_overwrites(mem); j++) {
       entity *ovw = get_entity_overwrites(mem, j);
-      //printf(" overwrites: "); DDME(ovw);
+      /*printf(" overwrites: "); DDME(ovw);*/
       /* Check whether ovw is member of one of tp's supertypes. If so,
-        the representation is correct. */
+     the representation is correct. */
       found = false;
       for (k = 0; k < get_class_n_supertypes(tp); k++) {
-       if (get_class_member_index(get_class_supertype(tp, k), ovw) >= 0) {
-         found = true;
-         break;
-       }
+    if (get_class_member_index(get_class_supertype(tp, k), ovw) >= 0) {
+      found = true;
+      break;
+    }
       }
       if (!found) {
-       DDMT(tp); DDME(mem);
-       assert(found && "overwrites an entity not contained in direct supertype");
-       return error_ent_not_cont;
+    DDMT(tp); DDME(mem);
+    assert(found && "overwrites an entity not contained in direct supertype");
+    return error_ent_not_cont;
       }
     }
 
@@ -43,6 +60,28 @@ static int check_class(type *tp) {
   return 0;
 }
 
+/**
+ * Check an array.
+ */
+static int check_array(type *tp) {
+  int i, n_dim = get_array_n_dimensions(tp);
+  for (i = 0; i < n_dim; ++i)
+    assert(has_array_lower_bound(tp, i) || has_array_upper_bound(tp, i));
+
+  return 0;
+}
+
+
+/**
+ * Check a primitive.
+ */
+static int check_primitive(type *tp) {
+  assert(is_mode(get_type_mode(tp)));
+
+  return 0;
+}
+
+
 /**
  * Checks a type.
  *
@@ -52,6 +91,10 @@ static int check_type(type *tp) {
   switch (get_type_tpop_code(tp)) {
   case tpo_class:
     return check_class(tp);
+  case tpo_array:
+    return check_array(tp);
+  case tpo_primitive:
+    return check_primitive(tp);
   default: break;
   }
   return 0;
@@ -72,6 +115,10 @@ static void on_irg_storage(ir_node *n, void *env) {
   struct myenv * myenv = env;
 
   myenv->res = node_is_in_irgs_storage(myenv->irg, n);
+
+  /* We also test whether the setting of the visited flag is legal. */
+  assert(get_irn_visited(n) <= get_irg_visited(myenv->irg) &&
+     "Visited flag of node is larger than that of corresponding irg.");
 }
 
 /**
@@ -92,22 +139,22 @@ static int constant_on_wrong_irg(ir_node *n) {
  * Check if constants node are NOT on the constant IR graph.
  */
 static int constants_on_wrong_irg(entity *ent) {
-  if (get_entity_variability(ent) == uninitialized) return 0;
+  if (get_entity_variability(ent) == variability_uninitialized) return 0;
 
   if (is_compound_entity(ent)) {
     int i;
     for (i = 0; i < get_compound_ent_n_values(ent); i++) {
       if (constant_on_wrong_irg(get_compound_ent_value(ent, i)))
-       return 1;
+    return 1;
     }
   } else {
-    /* Might not be set if entity belongs to a description. */
+    /* Might not be set if entity belongs to a description or is external allocated. */
     if (get_atomic_ent_value(ent))
       return constant_on_wrong_irg(get_atomic_ent_value(ent));
-    else
+    else if (get_entity_visibility(ent) != visibility_external_allocated)
       assert((is_class_type(get_entity_owner(ent)) &&
-             get_class_peculiarity(get_entity_owner(ent)) == description) &&
-            "Value in constant atomic entity not set.");
+          get_class_peculiarity(get_entity_owner(ent)) == peculiarity_description) &&
+         "Value in constant atomic entity not set.");
   }
   return 0;
 }
@@ -117,15 +164,36 @@ static int constants_on_wrong_irg(entity *ent) {
  * are build on the const irg graph.
  *
  * @return
- *     0       if no error encountered
- *     != 0    else
+ *  0   if no error encountered
+ *  != 0    else
  */
 static int check_entity(entity *ent) {
+  int rem_vpi;
+
+  current_ir_graph =  get_const_code_irg();
   if (constants_on_wrong_irg(ent)) {
     assert(0 && "Contants placed on wrong IRG");
     return error_const_on_wrong_irg;
   }
-  return 0;
+
+  rem_vpi = get_visit_pseudo_irgs();
+  set_visit_pseudo_irgs(1);
+  if ((get_entity_peculiarity(ent) == peculiarity_existent) &&
+      (get_entity_visibility(ent) != visibility_external_allocated) &&
+      (is_method_type(get_entity_type(ent)))                &&
+      (!get_entity_irg(ent) || !(is_ir_graph(get_entity_irg(ent))))) {
+    assert(0 && "Method ents with pec_exist must have an irg");
+    return error_existent_entity_without_irg;
+  }
+  set_visit_pseudo_irgs(rem_vpi);
+
+  if (get_entity_peculiarity(ent) == peculiarity_inherited) {
+    entity *impl = get_SymConst_entity(get_atomic_ent_value(ent));
+    assert(get_entity_peculiarity(impl) == peculiarity_existent &&
+           "inherited entities must have constant pointing to existent entity.");
+  }
+
+  return no_error;
 }
 
 /*
@@ -133,6 +201,7 @@ static int check_entity(entity *ent) {
  */
 static void check_tore(type_or_ent *tore, void *env) {
   int *res = env;
+  assert(tore);
   if (is_type(tore)) {
     *res = check_type((type *)tore);
   } else {