git
/
gcc
mirror of git://gcc.gnu.org/git/gcc.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

This patchset improves zero extend costs and code generation. 

When zero extending a 32-bit register, we emit a "mov", but currently
report the cost of the "mov" incorrectly.

In terms of speed, we currently say the cost is that of an extend
operation. But the cost of a "mov" is the cost of 1 instruction, so fix
that.

In terms of size, we currently say that the "mov" takes 0 instructions.
Fix it by changing it to 1.

Bootstrapped and tested on aarch64-none-elf.

    gcc/
	* config/aarch64/aarch64.c (aarch64_rtx_costs): Fix cost of zero extend.

From-SVN: r238820
This commit is contained in:
Kristina Martsenko 2016-07-28 14:29:28 +00:00 committed by Wilco Dijkstra
parent 89ac681e18
commit dde23f43f9
2 changed files with 10 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
gcc/ChangeLog
View File

@ -1,3 +1,7 @@
2016-07-28 Kristina Martsenko <kristina.martsenko@arm.com>
* config/aarch64/aarch64.c (aarch64_rtx_costs): Fix cost of zero extend.
2016-07-28 Wilco Dijkstra <wdijkstr@arm.com> 2016-07-28 Wilco Dijkstra <wdijkstr@arm.com>
* config/aarch64/aarch64.c (aarch64_pushwb_pair_reg): Rename. * config/aarch64/aarch64.c (aarch64_pushwb_pair_reg): Rename.

11
gcc/config/aarch64/aarch64.c
View File

@ -6803,11 +6803,12 @@ cost_plus:
{ {
int op_cost = rtx_cost (op0, VOIDmode, ZERO_EXTEND, 0, speed); int op_cost = rtx_cost (op0, VOIDmode, ZERO_EXTEND, 0, speed);
if (!op_cost && speed) /* If OP_COST is non-zero, then the cost of the zero extend
/* MOV. */ is effectively the cost of the inner operation. Otherwise
*cost += extra_cost->alu.extend; we have a MOV instruction and we take the cost from the MOV
else itself. This is true independently of whether we are
/* Free, the cost is that of the SI mode operation. */ optimizing for space or time. */
if (op_cost)
*cost = op_cost; *cost = op_cost;
return true; return true;