Re: [PATCH] Slight cost adjustment in SLSR

2017-12-14 Thread Richard Biener
On December 14, 2017 8:12:36 PM GMT+01:00, Bill Schmidt wrote: >Hi, > >While looking at PR83253, I noticed that the cost model for MULT_EXPR >replacement can be improved. Right now we use mult_by_coeff_cost to >determine the value of the possible replacement, but we use mul_cost >to determine th

[PATCH] Slight cost adjustment in SLSR

2017-12-14 Thread Bill Schmidt
Hi, While looking at PR83253, I noticed that the cost model for MULT_EXPR replacement can be improved. Right now we use mult_by_coeff_cost to determine the value of the possible replacement, but we use mul_cost to determine the savings from removing the original expression. This overcounts the s