+ if ((low != -8&& low != -4&& low != 0&& low != 4)
+ || !multiple_operation_profitable_p (false, 2, val))
should be moved into multiple_operation_profitable_p.
Since the code is identical - why not move it in there today or is that
what you are saying ? Ideally I think it would be better to move
this in
as a flag into the costs structure per core .
I'm not sure if it's good to move it into
multiple_operation_profitable_p since it's also called from some places.
Moving into it but not affecting other code needs more time thinking.
For now I just try to keep my patch as simple as possible to ease code
review. We can move it into multiple_operation_profitable_p later when
multiple_operation_profitable_p will be changed largely.