Commit 58ed456c by Richard Biener Committed by Richard Biener

re PR ipa/65270 (issues with merging memory accesses from different code paths)

2015-03-05  Richard Biener  <rguenther@suse.de>

	PR ipa/65270
	* ipa-icf-gimple.c (func_checker::compare_memory_operand):
	Compare dependence info.

From-SVN: r221204
parent b3406fa0
2015-03-05 Richard Biener <rguenther@suse.de> 2015-03-05 Richard Biener <rguenther@suse.de>
PR ipa/65270
* ipa-icf-gimple.c (func_checker::compare_memory_operand):
Compare dependence info.
2015-03-05 Richard Biener <rguenther@suse.de>
PR middle-end/65233 PR middle-end/65233
* ipa-polymorphic-call.c: Include tree-ssa-operands.h and * ipa-polymorphic-call.c: Include tree-ssa-operands.h and
tree-into-ssa.h. tree-into-ssa.h.
......
...@@ -305,6 +305,23 @@ func_checker::compare_memory_operand (tree t1, tree t2) ...@@ -305,6 +305,23 @@ func_checker::compare_memory_operand (tree t1, tree t2)
get_object_alignment_1 (b2, &align2, &tem); get_object_alignment_1 (b2, &align2, &tem);
if (align1 != align2) if (align1 != align2)
return return_false_with_msg ("different access alignment"); return return_false_with_msg ("different access alignment");
/* Similarly we have to compare dependence info where equality
tells us we are safe (even some unequal values would be safe
but then we have to maintain a map of bases and cliques). */
unsigned short clique1 = 0, base1 = 0, clique2 = 0, base2 = 0;
if (TREE_CODE (b1) == MEM_REF)
{
clique1 = MR_DEPENDENCE_CLIQUE (b1);
base1 = MR_DEPENDENCE_BASE (b1);
}
if (TREE_CODE (b2) == MEM_REF)
{
clique2 = MR_DEPENDENCE_CLIQUE (b2);
base2 = MR_DEPENDENCE_BASE (b2);
}
if (clique1 != clique2 || base1 != base2)
return return_false_with_msg ("different dependence info");
} }
return compare_operand (t1, t2); return compare_operand (t1, t2);
......
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