On 06/21/2016 09:50 AM, James Greenhalgh wrote:
Hi,
This transformation tries two cost models, one estimating the number
of insns to use, one estimating the RTX cost of the transformed sequence.
This is inconsistent with the other cost models used in ifcvt.c and
unneccesary - eliminate the second cost model.
Thanks,
James
---
2016-06-21 James Greenhalgh <james.greenha...@arm.com>
* ifcvt.c (noce_try_store_flag_mask): Delete redundant cost model.
OK.
At this point I think the series is fully ack'd. Given folks have had
nearly a month to object to the overall direction, I think you should go
ahead and commit. We can deal with any fallout at the target maintainer
level.
jeff