This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [4.0 and mainline] Fix multiplication by constant expansion


On Sun, Jan 01, 2006 at 09:39:18PM +0100, Jan Hubicka wrote:
>   	  if (shiftadd_cost[mode][m] < op_cost)
> ! 	    op_cost = shiftadd_cost[mode][m];
>   	  else
>   	    op_latency = add_cost[mode];
>   
>   	  new_limit.cost = best_cost.cost - op_cost;
> ! 	  new_limit.latency = best_cost.latency - op_cost;

I don't like you mixing units like this in the last line.

> + 	  /* Because we want to preffer combined lea over disjoint
> + 	     operations, we are interested in sollutions having equal

Please spell-check all your comments.  Lots of doubled letters.


r~


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]