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: [PATCH] Reorganize rs6000_rtx_costs


>>>>> Roger Sayle writes:

> There should be no functional/parameter differences with this patch
> other than to report the size of multiplication and and division
> operations as a single instruction when optimizing for size.  A
> minor difference to the x86 backend is that I introduce two "size"
> tables, one for 32bit code and the other for 64bit code, selected
> using TARGET_POWERPC64.  This will allow us to record the fact that
> a 64-bit multiplication is significantly shorter on processors
> when compiling for a 64-bit CPU, than it is for a 32-bit CPU.

	Do you mean that the table already records this information, or
that it can record the information?  size32_cost and size64_cost both
uniformly contain the value 1, so I do not see how this provides any
distinction.

	Also, the original code returned COSTS_N_INSNS(2), not
COSTS_N_INSNS(1), for optimize_size.  Any reason for the change?

David


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