From: Christoph Mallon <christoph.mallon@gmx.de>
Date: Mon, 4 Aug 2008 18:45:22 +0000 (+0000)
Subject: Using an obstack is slightly overkill.
X-Git-Url: http://nsz.repo.hu/git/?a=commitdiff_plain;h=c64dcb13abaab1987775213532cc0fa9542e1874;p=libfirm

Using an obstack is slightly overkill.

[r20970]
---

diff --git a/ir/lower/lower_switch.c b/ir/lower/lower_switch.c
index 1c7945542..301cba462 100644
--- a/ir/lower/lower_switch.c
+++ b/ir/lower/lower_switch.c
@@ -41,7 +41,6 @@
 
 typedef struct walk_env {
 	unsigned         spare_size;        /**< the allowed spare size for table switches */
-	struct obstack   obst;              /**< the obstack where data is allocated on */
 	int              changed;           /**< indicates whether a change was performed */
 } walk_env_t;
 
@@ -207,7 +206,7 @@ static void find_cond_nodes(ir_node *block, void *ctx)
 	 */
 
 	numcases = get_irn_n_outs(cond) - 1;      // does not contain default case
-	cases    = obstack_alloc(&env->obst, numcases * sizeof(*cases));
+	NEW_ARR_A(case_data_t, cases, numcases);
 
 	default_pn = get_Cond_defaultProj(cond);
 	ifcas_env.sel = sel;
@@ -239,8 +238,6 @@ static void find_cond_nodes(ir_node *block, void *ctx)
 
 	/* Connect new default case users */
 	set_irn_in(defblock, ifcas_env.defindex, ifcas_env.defusers);
-
-	obstack_free(&env->obst, cases);
 }
 
 /**
@@ -258,7 +255,6 @@ void lower_switch(ir_graph *irg, unsigned spare_size)
 
 	current_ir_graph = irg;
 
-	obstack_init(&env.obst);
 	env.spare_size = spare_size;
 
 	remove_critical_cf_edges(irg);
@@ -274,6 +270,5 @@ void lower_switch(ir_graph *irg, unsigned spare_size)
 		set_irg_loopinfo_inconsistent(irg);
 	}
 
-	obstack_free(&env.obst, NULL);
 	current_ir_graph = rem;
 }