besched: Remove the unused/unnecessary functions sched_has_next() and sched_has_prev().
authorChristoph Mallon <christoph.mallon@gmx.de>
Sat, 24 Nov 2012 20:28:19 +0000 (21:28 +0100)
committerChristoph Mallon <christoph.mallon@gmx.de>
Sat, 24 Nov 2012 21:09:25 +0000 (22:09 +0100)
ir/be/beinfo.c
ir/be/besched.h
ir/be/sparc/sparc_finish.c

index 332173e..bc6b5a3 100644 (file)
@@ -155,13 +155,15 @@ static void sched_edge_hook(FILE *F, const ir_node *irn)
        if (get_irn_irg(irn)->be_data == NULL)
                return;
 
-       if (sched_is_scheduled(irn) && sched_has_prev(irn) && !is_Block(irn)) {
-               ir_node *prev = sched_prev(irn);
-               fprintf(F, "edge:{sourcename: ");
-               print_nodeid(F, irn);
-               fprintf(F, " targetname: ");
-               print_nodeid(F, prev);
-               fprintf(F, " color:magenta}\n");
+       if (sched_is_scheduled(irn) && !is_Block(irn)) {
+               ir_node *const prev = sched_prev(irn);
+               if (!sched_is_begin(prev)) {
+                       fprintf(F, "edge:{sourcename: ");
+                       print_nodeid(F, irn);
+                       fprintf(F, " targetname: ");
+                       print_nodeid(F, prev);
+                       fprintf(F, " color:magenta}\n");
+               }
        }
 }
 
index b080b0b..bb9694f 100644 (file)
@@ -74,30 +74,6 @@ static inline bool sched_is_begin(const ir_node *node)
        return is_Block(node);
 }
 
-/**
- * Check, if an ir_node has a scheduling successor.
- * @param irn The ir node.
- * @return 1, if the node has a scheduling successor, 0 if not.
- */
-static inline bool sched_has_next(const ir_node *irn)
-{
-       const sched_info_t *info  = get_irn_sched_info(irn);
-       const ir_node      *block = is_Block(irn) ? irn : get_nodes_block(irn);
-       return info->next != block;
-}
-
-/**
- * Check, if an ir_node has a scheduling predecessor.
- * @param irn The ir node.
- * @return 1, if the node has a scheduling predecessor, 0 if not.
- */
-static inline bool sched_has_prev(const ir_node *irn)
-{
-       const sched_info_t *info  = get_irn_sched_info(irn);
-       const ir_node      *block = is_Block(irn) ? irn : get_nodes_block(irn);
-       return info->prev != block;
-}
-
 /**
  * Get the scheduling successor of a node.
  * @param irn The node.
index 4c34288..65a08d4 100644 (file)
@@ -519,11 +519,12 @@ static void peephole_sparc_RestoreZero(ir_node *node)
         * (and therefore after code selection).
         */
        int n_tries = 10; /* limit our search */
-       ir_node *schedpoint = node;
 
-       while (sched_has_prev(schedpoint)) {
+       for (ir_node *schedpoint = node;;) {
                const arch_register_t *reg;
                schedpoint = sched_prev(schedpoint);
+               if (sched_is_begin(schedpoint))
+                       break;
 
                if (--n_tries == 0)
                        break;
@@ -591,9 +592,9 @@ static void finish_sparc_Return(ir_node *node)
        /* see that there is no code between Return and restore, if there is move
         * it in front of the restore */
        while (true) {
-               if (!sched_has_prev(schedpoint))
-                       return;
                schedpoint = sched_prev(schedpoint);
+               if (sched_is_begin(schedpoint))
+                       return;
                if (is_sparc_Restore(schedpoint) || is_sparc_RestoreZero(schedpoint))
                        break;
        }