Commit 800cb72a by Jakub Jelinek Committed by Jakub Jelinek

ifcvt.c (bb_ok_for_noce_convert_multiple_sets): Return false when count is…

ifcvt.c (bb_ok_for_noce_convert_multiple_sets): Return false when count is incremented above limit...

	* ifcvt.c (bb_ok_for_noce_convert_multiple_sets): Return false
	when count is incremented above limit, don't analyze further
	insns afterwards.

From-SVN: r233050
parent 9a7013f8
2016-02-01 Jakub Jelinek <jakub@redhat.com> 2016-02-01 Jakub Jelinek <jakub@redhat.com>
* ifcvt.c (bb_ok_for_noce_convert_multiple_sets): Return false
when count is incremented above limit, don't analyze further
insns afterwards.
* omp-low.c (oacc_parse_default_dims): Avoid * omp-low.c (oacc_parse_default_dims): Avoid
-Wsign-compare warning, make sure value fits into int -Wsign-compare warning, make sure value fits into int
rather than just unsigned int. rather than just unsigned int.
......
...@@ -3286,14 +3286,12 @@ bb_ok_for_noce_convert_multiple_sets (basic_block test_bb, ...@@ -3286,14 +3286,12 @@ bb_ok_for_noce_convert_multiple_sets (basic_block test_bb,
if (!can_conditionally_move_p (GET_MODE (dest))) if (!can_conditionally_move_p (GET_MODE (dest)))
return false; return false;
++count;
}
/* FORNOW: Our cost model is a count of the number of instructions we /* FORNOW: Our cost model is a count of the number of instructions we
would if-convert. This is suboptimal, and should be improved as part would if-convert. This is suboptimal, and should be improved as part
of a wider rework of branch_cost. */ of a wider rework of branch_cost. */
if (count > limit) if (++count > limit)
return false; return false;
}
return count > 1; return count > 1;
} }
......
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