Commit dbaa912c by Richard Guenther Committed by Richard Biener

re PR middle-end/48270 (New test failures)

2011-03-24  Richard Guenther  <rguenther@suse.de>

	PR tree-optimization/48270
	* tree-ssa-phiopt.c (cond_if_else_store_replacement): Do
	not free datarefs before ddrs.

From-SVN: r171394
parent 1d60cc55
2011-03-24 Richard Guenther <rguenther@suse.de>
PR tree-optimization/48270
* tree-ssa-phiopt.c (cond_if_else_store_replacement): Do
not free datarefs before ddrs.
2011-03-24 Eric Botcazou <ebotcazou@adacore.com> 2011-03-24 Eric Botcazou <ebotcazou@adacore.com>
* tree-sra.c (build_ref_for_offset): Strip useless type conversions * tree-sra.c (build_ref_for_offset): Strip useless type conversions
......
...@@ -1494,8 +1494,6 @@ cond_if_else_store_replacement (basic_block then_bb, basic_block else_bb, ...@@ -1494,8 +1494,6 @@ cond_if_else_store_replacement (basic_block then_bb, basic_block else_bb,
else_ddrs = VEC_alloc (ddr_p, heap, 1); else_ddrs = VEC_alloc (ddr_p, heap, 1);
compute_all_dependences (then_datarefs, &then_ddrs, NULL, false); compute_all_dependences (then_datarefs, &then_ddrs, NULL, false);
compute_all_dependences (else_datarefs, &else_ddrs, NULL, false); compute_all_dependences (else_datarefs, &else_ddrs, NULL, false);
free_data_refs (then_datarefs);
free_data_refs (else_datarefs);
blocks[0] = then_bb; blocks[0] = then_bb;
blocks[1] = else_bb; blocks[1] = else_bb;
blocks[2] = join_bb; blocks[2] = join_bb;
...@@ -1517,6 +1515,8 @@ cond_if_else_store_replacement (basic_block then_bb, basic_block else_bb, ...@@ -1517,6 +1515,8 @@ cond_if_else_store_replacement (basic_block then_bb, basic_block else_bb,
{ {
free_dependence_relations (then_ddrs); free_dependence_relations (then_ddrs);
free_dependence_relations (else_ddrs); free_dependence_relations (else_ddrs);
free_data_refs (then_datarefs);
free_data_refs (else_datarefs);
VEC_free (gimple, heap, then_stores); VEC_free (gimple, heap, then_stores);
VEC_free (gimple, heap, else_stores); VEC_free (gimple, heap, else_stores);
return false; return false;
...@@ -1539,6 +1539,8 @@ cond_if_else_store_replacement (basic_block then_bb, basic_block else_bb, ...@@ -1539,6 +1539,8 @@ cond_if_else_store_replacement (basic_block then_bb, basic_block else_bb,
{ {
free_dependence_relations (then_ddrs); free_dependence_relations (then_ddrs);
free_dependence_relations (else_ddrs); free_dependence_relations (else_ddrs);
free_data_refs (then_datarefs);
free_data_refs (else_datarefs);
VEC_free (gimple, heap, then_stores); VEC_free (gimple, heap, then_stores);
VEC_free (gimple, heap, else_stores); VEC_free (gimple, heap, else_stores);
return false; return false;
...@@ -1556,6 +1558,8 @@ cond_if_else_store_replacement (basic_block then_bb, basic_block else_bb, ...@@ -1556,6 +1558,8 @@ cond_if_else_store_replacement (basic_block then_bb, basic_block else_bb,
free_dependence_relations (then_ddrs); free_dependence_relations (then_ddrs);
free_dependence_relations (else_ddrs); free_dependence_relations (else_ddrs);
free_data_refs (then_datarefs);
free_data_refs (else_datarefs);
VEC_free (gimple, heap, then_stores); VEC_free (gimple, heap, then_stores);
VEC_free (gimple, heap, else_stores); VEC_free (gimple, heap, else_stores);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment