On 11/24/2016 03:21 PM, Segher Boessenkool wrote:
Combine uses insn_rtx_cost extensively. I have tried to change it to use the full rtx cost, not just the source cost, a few times before, and it always only regressed. Part of it is that most ports' cost calculations are, erm, not so great -- every target we care about needs fixes.Let's please not try this in stage 3.
It got approved and committed. Do you want me to revert it now or wait for obvious signs of fallout?
Bernd