On 06/21/2016 09:50 AM, James Greenhalgh wrote:
Hi, This patch is rewrites the cost model for bb_ok_for_noce_multiple_sets to use the max_seq_cost heuristic added in earlier patch revisions. As with the previous patch, I've used the new parameters to ensure that the testsuite is still testing the functionality rather than relying on the target setting the costs appropriately. Thanks, James --- gcc/ 2016-06-21 James Greenhalgh <james.greenha...@arm.com> * ifcvt.c (noce_convert_multiple sets): Move cost model to here, check the sequence cost after constructing the converted sequence. (bb_of_for_noce_convert_multiple_sets): Move cost model. gcc/testsuite/ 2016-06-21 James Greenhalgh <james.greenha...@arm.com> * gcc.dg/ifcvt-4.c: Use parameter to guide if-conversion heuristics. * gcc.dg/ifcvt-5.c: Use parameter to guide if-conversion heuristics.
OK. jeff