fixed some warnings
[libfirm] / ir / ir / iredges.c
index 34a3640..2b6e19d 100644 (file)
@@ -1,32 +1,36 @@
 /*
- * Project:     libFIRM
- * File name:   ir/ir/iredges.c
- * Purpose:     Always available outs.
- * Author:      Sebastian Hack
- * Modified by: Michael Beck, Andreas Schoesser
- * Created:     14.1.2005
- * CVS-ID:      $Id$
- * Copyright:   (c) 1998-2006 Universität Karlsruhe
- * Licence:     This file protected by GPL -  GNU GENERAL PUBLIC LICENSE.
+ * Copyright (C) 1995-2007 University of Karlsruhe.  All right reserved.
+ *
+ * This file is part of libFirm.
+ *
+ * This file may be distributed and/or modified under the terms of the
+ * GNU General Public License version 2 as published by the Free Software
+ * Foundation and appearing in the file LICENSE.GPL included in the
+ * packaging of this file.
+ *
+ * Licensees holding valid libFirm Professional Edition licenses may use
+ * this file in accordance with the libFirm Commercial License.
+ * Agreement provided with the Software.
+ *
+ * This file is provided AS IS with NO WARRANTY OF ANY KIND, INCLUDING THE
+ * WARRANTY OF DESIGN, MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE.
  */
 
 /**
- * Always available outs.
- * @author Sebastian Hack
- * @date 14.1.2005
+ * @file
+ * @brief   Always available outs.
+ * @author  Sebastian Hack, Michael Beck, Andreas Schoesser
+ * @date    14.1.2005
+ * @version $Id$
+ * @summary
+ *   This are out-edges (also called def-use edges) that are dynamically
+ *   updated as the graph changes.
  */
-
 #ifdef HAVE_CONFIG_H
 #include "config.h"
 #endif
 
-#ifdef HAVE_ALLOCA_H
-#include <alloca.h>
-#endif
-#ifdef HAVE_MALLOC_H
-#include <malloc.h>
-#endif
-
 #include "irnode_t.h"
 #include "iropt_t.h"
 #include "iredgekinds.h"
 #include "debug.h"
 #include "set.h"
 #include "bitset.h"
+#include "xmalloc.h"
 
 /**
-* A function that allows for setting an edge.
-* This abstraction is necessary since different edge kind have
-* different methods of setting edges.
-*/
+ * A function that allows for setting an edge.
+ * This abstraction is necessary since different edge kind have
+ * different methods of setting edges.
+ */
 typedef void (set_edge_func_t)(ir_node *src, int pos, ir_node *tgt);
 
 typedef int (get_edge_src_arity_func_t)(const ir_node *src);
@@ -53,8 +58,8 @@ typedef int (get_edge_src_first_func_t)(const ir_node *src);
 typedef ir_node *(get_edge_src_n_func_t)(const ir_node *src, int pos);
 
 /**
-* Additional data for an edge kind.
-*/
+ * Additional data for an edge kind.
+ */
 typedef struct {
        const char                *name;
        set_edge_func_t           *set_edge;
@@ -65,6 +70,7 @@ typedef struct {
 
 static int get_zero(const ir_node *irn)
 {
+       (void) irn;
        return 0;
 }
 
@@ -108,8 +114,10 @@ static int edges_private_size = 0;
  */
 static int edges_dbg = 0;
 
+#ifdef DEBUG_libfirm
 /* a static variable holding the last number assigned to a new edge */
 static long last_edge_num = -1;
+#endif
 
 static INLINE long edge_get_id(const ir_edge_t *e) {
 #ifdef DEBUG_libfirm
@@ -162,6 +170,7 @@ static int edge_cmp(const void *p1, const void *p2, size_t len)
 {
        const ir_edge_t *e1 = p1;
        const ir_edge_t *e2 = p2;
+       (void) len;
 
        if(e1->src != e2->src)
                return 1;
@@ -254,7 +263,7 @@ static INLINE void vrfy_list_head(ir_node *irn, ir_edge_kind_t kind) {
                        fprintf(stderr, "- at list entry %d\n", num);
                        if (edge->invalid)
                                fprintf(stderr, "- edge(%ld) is invalid\n", edge_get_id(edge));
-                       if (edge->src);
+                       if (edge->src)
                                ir_fprintf(stderr, "- edge(%ld) %+F(%d)\n", edge_get_id(edge), edge->src, edge->pos);
                        err = 1;
                        break;
@@ -368,6 +377,7 @@ void edges_notify_edge_kind(ir_node *src, int pos, ir_node *tgt,
                edge_change_cnt(tgt, kind, +1);
        } /* else */
 
+#ifndef DEBUG_libfirm
        /* verify list heads */
        if (edges_dbg) {
                if (tgt)
@@ -375,6 +385,7 @@ void edges_notify_edge_kind(ir_node *src, int pos, ir_node *tgt,
                if (old_tgt)
                        vrfy_list_head(old_tgt, kind);
        }
+#endif
 
        DBG((dbg, LEVEL_5, "announce out edge: %+F %d-> %+F(%+F): %s\n", src, pos, tgt, old_tgt, msg));
 }
@@ -487,8 +498,6 @@ void edges_activate_kind(ir_graph *irg, ir_edge_kind_t kind)
 
        info->activated = 1;
        edges_init_graph_kind(irg, kind);
-       //irg_walk_graph(irg, init_lh_walker, build_edges_walker, &w);
-       inc_irg_visited(irg);
        irg_walk_anchors(irg, init_lh_walker, build_edges_walker, &w);
        visit_all_identities(irg, visitor, &w);
 }
@@ -643,6 +652,7 @@ static void clear_links(ir_node *irn, void *env) {
 static void count_user(ir_node *irn, void *env) {
        int i;
        int first;
+       (void) env;
 
        first = get_irn_first(irn);
        for (i = get_irn_arity(irn) - 1; i >= first; --i) {
@@ -663,6 +673,7 @@ static void verify_edge_counter(ir_node *irn, void *env) {
        int                    list_cnt;
        int                    ref_cnt;
        int                    edge_cnt;
+       unsigned long          idx;
        const struct list_head *head;
        const struct list_head *pos;
 
@@ -671,7 +682,7 @@ static void verify_edge_counter(ir_node *irn, void *env) {
 
        bs       = get_irn_link(irn);
        list_cnt = 0;
-       ref_cnt  = bitset_popcnt(bs);
+       ref_cnt = 0;
        edge_cnt = _get_irn_edge_info(irn, EDGE_KIND_NORMAL)->out_count;
        head     = _get_irn_outs_head(irn, EDGE_KIND_NORMAL);
 
@@ -680,6 +691,21 @@ static void verify_edge_counter(ir_node *irn, void *env) {
                list_cnt++;
        }
 
+       /* check all nodes that reference us and count edges that point number
+        * of ins that actually point to us */
+       ref_cnt = 0;
+       bitset_foreach(bs, idx) {
+               int i, arity;
+               ir_node *src = get_idx_irn(w->irg, idx);
+
+               arity = get_irn_arity(src);
+               for(i = 0; i < arity; ++i) {
+                       ir_node *in = get_irn_n(src, i);
+                       if(in == irn)
+                               ref_cnt++;
+               }
+       }
+
        if (edge_cnt != list_cnt) {
                w->problem_found = 1;
                ir_fprintf(stderr, "Edge Verifier: edge count is %d, but %d edge(s) are recorded in list at %+F\n",
@@ -687,12 +713,12 @@ static void verify_edge_counter(ir_node *irn, void *env) {
        }
 
        if (ref_cnt != list_cnt) {
-               unsigned long idx;
-
                w->problem_found = 1;
-               ir_fprintf(stderr, "Edge Verifier: %+F reachable by %d node(s), but %d edge(s) recorded in list\n",
+               ir_fprintf(stderr, "Edge Verifier: %+F reachable by %d node(s), but the list contains %d edge(s)\n",
                        irn, ref_cnt, list_cnt);
 
+               /* Matze: buggy if a node has multiple ins pointing at irn */
+#if 0
                list_for_each(pos, head) {
                        ir_edge_t *edge = list_entry(pos, ir_edge_t, list);
                        bitset_flip(bs, get_irn_idx(edge->src));
@@ -709,6 +735,7 @@ static void verify_edge_counter(ir_node *irn, void *env) {
                        ir_fprintf(stderr, " %+F", src);
                }
                fprintf(stderr, "\n");
+#endif
        }
 
        bitset_free(bs);
@@ -721,9 +748,6 @@ int edges_verify(ir_graph *irg) {
        struct build_walker w;
        int    problem_found = 0;
 
-       if (! edges_dbg)
-               return 0;
-
        /* verify normal edges only */
        problem_found  = edges_verify_kind(irg, EDGE_KIND_NORMAL);
 
@@ -732,9 +756,7 @@ int edges_verify(ir_graph *irg) {
        w.problem_found = 0;
 
        /* verify counter */
-       inc_irg_visited(irg);
        irg_walk_anchors(irg, clear_links, count_user, &w);
-       inc_irg_visited(irg);
        irg_walk_anchors(irg, NULL, verify_edge_counter, &w);
 
        return problem_found ? 1 : w.problem_found;
@@ -816,3 +838,38 @@ void dump_all_out_edges(ir_node *irn)
                }
        }
 }
+
+static void irg_block_edges_walk2(ir_node *bl,
+                                irg_walk_func *pre, irg_walk_func *post,
+                                void *env) {
+       const ir_edge_t *edge, *next;
+
+       if (Block_not_block_visited(bl)) {
+               mark_Block_block_visited(bl);
+
+               if (pre)
+                       pre(bl, env);
+
+               foreach_out_edge_kind_safe(bl, edge, next, EDGE_KIND_BLOCK) {
+                       /* find the corresponding successor block. */
+                       ir_node *pred = get_edge_src_irn(edge);
+                       irg_block_edges_walk2(pred, pre, post, env);
+               }
+
+               if (post)
+                       post(bl, env);
+       }
+}
+
+/* Walks only over Block nodes in the graph.  Has it's own visited
+   flag, so that it can be interleaved with the other walker.         */
+void irg_block_edges_walk(ir_node *node,
+                          irg_walk_func *pre, irg_walk_func *post,
+                          void *env) {
+
+       assert(edges_activated(current_ir_graph));
+       assert(is_Block(node));
+
+       inc_irg_block_visited(current_ir_graph);
+       irg_block_edges_walk2(node, pre, post, env);
+}