mirror of
https://gitlab.com/freepascal.org/fpc/source.git
synced 2025-08-12 15:49:44 +02:00
+ MIPS: implemented more peephole optimizations.
git-svn-id: trunk@28536 -
This commit is contained in:
parent
d9a7d28838
commit
1e11e34f42
@ -28,15 +28,19 @@ unit aoptcpu;
|
|||||||
Interface
|
Interface
|
||||||
|
|
||||||
uses
|
uses
|
||||||
cgbase, cpubase, aoptobj, aoptcpub, aopt, aasmtai;
|
cgbase, cpubase, aoptobj, aoptcpub, aopt, aasmtai, aasmcpu;
|
||||||
|
|
||||||
Type
|
Type
|
||||||
|
TAsmOpSet = set of TAsmOp;
|
||||||
|
|
||||||
TCpuAsmOptimizer = class(TAsmOptimizer)
|
TCpuAsmOptimizer = class(TAsmOptimizer)
|
||||||
|
function RegModifiedByInstruction(Reg: TRegister; p1: tai): boolean; override;
|
||||||
function GetNextInstructionUsingReg(Current: tai;
|
function GetNextInstructionUsingReg(Current: tai;
|
||||||
var Next: tai; reg: TRegister): Boolean;
|
var Next: tai; reg: TRegister): Boolean;
|
||||||
function RegUsedAfterInstruction(reg: Tregister; p: tai;
|
function RegUsedAfterInstruction(reg: Tregister; p: tai;
|
||||||
var AllUsedRegs: TAllUsedRegs): Boolean;
|
var AllUsedRegs: TAllUsedRegs): Boolean;
|
||||||
function TryRemoveMov(var p: tai; opcode: TAsmOp): boolean;
|
function TryRemoveMov(var p: tai; opcode: TAsmOp): boolean;
|
||||||
|
function TryRemoveMovBeforeStore(var p: tai; next: taicpu; const storeops: TAsmOpSet): boolean;
|
||||||
function PeepHoleOptPass1Cpu(var p: tai): boolean; override;
|
function PeepHoleOptPass1Cpu(var p: tai): boolean; override;
|
||||||
procedure PeepHoleOptPass2; override;
|
procedure PeepHoleOptPass2; override;
|
||||||
End;
|
End;
|
||||||
@ -44,7 +48,7 @@ unit aoptcpu;
|
|||||||
Implementation
|
Implementation
|
||||||
|
|
||||||
uses
|
uses
|
||||||
globals,aasmbase,aasmcpu,cpuinfo,verbose;
|
globals,aasmbase,cpuinfo,verbose;
|
||||||
|
|
||||||
|
|
||||||
function MatchInstruction(const instr: tai; const op: TAsmOp): boolean;
|
function MatchInstruction(const instr: tai; const op: TAsmOp): boolean;
|
||||||
@ -170,6 +174,20 @@ unit aoptcpu;
|
|||||||
end;
|
end;
|
||||||
|
|
||||||
|
|
||||||
|
function TCpuAsmOptimizer.RegModifiedByInstruction(Reg: TRegister; p1: tai): boolean;
|
||||||
|
var
|
||||||
|
i : Longint;
|
||||||
|
begin
|
||||||
|
result:=false;
|
||||||
|
for i:=0 to taicpu(p1).ops-1 do
|
||||||
|
if (taicpu(p1).oper[i]^.typ=top_reg) and (taicpu(p1).oper[i]^.reg=Reg) and (taicpu(p1).spilling_get_operation_type(i) in [operand_write,operand_readwrite]) then
|
||||||
|
begin
|
||||||
|
result:=true;
|
||||||
|
exit;
|
||||||
|
end;
|
||||||
|
end;
|
||||||
|
|
||||||
|
|
||||||
function TCpuAsmOptimizer.GetNextInstructionUsingReg(Current: tai;
|
function TCpuAsmOptimizer.GetNextInstructionUsingReg(Current: tai;
|
||||||
var Next: tai; reg: TRegister): Boolean;
|
var Next: tai; reg: TRegister): Boolean;
|
||||||
begin
|
begin
|
||||||
@ -261,6 +279,23 @@ unit aoptcpu;
|
|||||||
end;
|
end;
|
||||||
|
|
||||||
|
|
||||||
|
function TCpuAsmOptimizer.TryRemoveMovBeforeStore(var p: tai; next: taicpu; const storeops: TAsmOpSet): boolean;
|
||||||
|
begin
|
||||||
|
result:=(next.opcode in storeops) and
|
||||||
|
MatchOperand(next.oper[0]^,taicpu(p).oper[0]^.reg) and
|
||||||
|
{ Ry cannot be modified between move and store }
|
||||||
|
(not RegModifiedBetween(taicpu(p).oper[1]^.reg,p,next)) and
|
||||||
|
Assigned(FindRegDealloc(taicpu(p).oper[0]^.reg,tai(next.next)));
|
||||||
|
if result then
|
||||||
|
begin
|
||||||
|
next.loadreg(0,taicpu(p).oper[1]^.reg);
|
||||||
|
asml.remove(p);
|
||||||
|
p.free;
|
||||||
|
p:=next;
|
||||||
|
end;
|
||||||
|
end;
|
||||||
|
|
||||||
|
|
||||||
function TCpuAsmOptimizer.PeepHoleOptPass1Cpu(var p: tai): boolean;
|
function TCpuAsmOptimizer.PeepHoleOptPass1Cpu(var p: tai): boolean;
|
||||||
var
|
var
|
||||||
next,next2: tai;
|
next,next2: tai;
|
||||||
@ -385,6 +420,92 @@ unit aoptcpu;
|
|||||||
TryRemoveMov(p,A_MOVE);
|
TryRemoveMov(p,A_MOVE);
|
||||||
end;
|
end;
|
||||||
|
|
||||||
|
A_MOV_S:
|
||||||
|
begin
|
||||||
|
if GetNextInstructionUsingReg(p,next,taicpu(p).oper[0]^.reg) and
|
||||||
|
(next.typ=ait_instruction) then
|
||||||
|
begin
|
||||||
|
if TryRemoveMovBeforeStore(p,taicpu(next),[A_SWC1]) then
|
||||||
|
{ optimization successful };
|
||||||
|
end;
|
||||||
|
end;
|
||||||
|
|
||||||
|
A_MOV_D:
|
||||||
|
begin
|
||||||
|
if GetNextInstructionUsingReg(p,next,taicpu(p).oper[0]^.reg) and
|
||||||
|
(next.typ=ait_instruction) then
|
||||||
|
begin
|
||||||
|
if TryRemoveMovBeforeStore(p,taicpu(next),[A_SDC1]) then
|
||||||
|
{ optimization successful };
|
||||||
|
end;
|
||||||
|
end;
|
||||||
|
|
||||||
|
A_MOVE:
|
||||||
|
begin
|
||||||
|
if GetNextInstructionUsingReg(p,next,taicpu(p).oper[0]^.reg) and
|
||||||
|
(next.typ=ait_instruction) then
|
||||||
|
begin
|
||||||
|
{ MOVE Rx,Ry; store Rx,(ref); dealloc Rx ==> store Ry,(ref) }
|
||||||
|
if TryRemoveMovBeforeStore(p,taicpu(next),[A_SB,A_SH,A_SW]) then
|
||||||
|
{ optimization successful }
|
||||||
|
{ MOVE Rx,Ry; opcode Rx,Rx,any ==> opcode Rx,Ry,any
|
||||||
|
MOVE Rx,Ry; opcode Rx,Rz,Rx ==> opcode Rx,Rz,Ry }
|
||||||
|
else if (taicpu(next).opcode in [A_ADD,A_ADDU,A_ADDI,A_ADDIU,A_SUB,A_SUBU]) and
|
||||||
|
MatchOperand(taicpu(next).oper[0]^,taicpu(p).oper[0]^.reg) and
|
||||||
|
(not RegModifiedBetween(taicpu(p).oper[1]^.reg,p,next)) then
|
||||||
|
begin
|
||||||
|
if MatchOperand(taicpu(next).oper[1]^,taicpu(p).oper[0]^.reg) then
|
||||||
|
begin
|
||||||
|
taicpu(next).loadreg(1,taicpu(p).oper[1]^.reg);
|
||||||
|
asml.remove(p);
|
||||||
|
p.free;
|
||||||
|
p:=next;
|
||||||
|
end
|
||||||
|
{ TODO: if Ry=NR_R0, this effectively changes instruction into MOVE,
|
||||||
|
providing further optimization possibilities }
|
||||||
|
else if MatchOperand(taicpu(next).oper[2]^,taicpu(p).oper[0]^.reg) then
|
||||||
|
begin
|
||||||
|
taicpu(next).loadreg(2,taicpu(p).oper[1]^.reg);
|
||||||
|
asml.remove(p);
|
||||||
|
p.free;
|
||||||
|
p:=next;
|
||||||
|
end;
|
||||||
|
end
|
||||||
|
{ MOVE Rx,Ry; opcode Rz,Rx,any; dealloc Rx ==> opcode Rz,Ry,any }
|
||||||
|
else if (taicpu(next).opcode in [A_ADD,A_ADDU,A_ADDI,A_ADDIU,A_SUB,A_SUBU,A_SLT,A_SLTU]) and
|
||||||
|
MatchOperand(taicpu(next).oper[1]^,taicpu(p).oper[0]^.reg) and
|
||||||
|
(not RegModifiedBetween(taicpu(p).oper[1]^.reg,p,next)) and
|
||||||
|
Assigned(FindRegDealloc(taicpu(p).oper[0]^.reg,tai(next.next))) then
|
||||||
|
begin
|
||||||
|
taicpu(next).loadreg(1,taicpu(p).oper[1]^.reg);
|
||||||
|
asml.remove(p);
|
||||||
|
p.free;
|
||||||
|
p:=next;
|
||||||
|
end
|
||||||
|
{ MULT[U] must be handled separately }
|
||||||
|
else if (taicpu(next).opcode in [A_MULT,A_MULTU]) and
|
||||||
|
(not RegModifiedBetween(taicpu(p).oper[1]^.reg,p,next)) and
|
||||||
|
Assigned(FindRegDealloc(taicpu(p).oper[0]^.reg,tai(next.next))) then
|
||||||
|
begin
|
||||||
|
if MatchOperand(taicpu(next).oper[0]^,taicpu(p).oper[0]^.reg) then
|
||||||
|
begin
|
||||||
|
taicpu(next).loadreg(0,taicpu(p).oper[1]^.reg);
|
||||||
|
asml.remove(p);
|
||||||
|
p.free;
|
||||||
|
p:=next;
|
||||||
|
end
|
||||||
|
else if MatchOperand(taicpu(next).oper[1]^,taicpu(p).oper[0]^.reg) then
|
||||||
|
begin
|
||||||
|
taicpu(next).loadreg(1,taicpu(p).oper[1]^.reg);
|
||||||
|
asml.remove(p);
|
||||||
|
p.free;
|
||||||
|
p:=next;
|
||||||
|
end;
|
||||||
|
end;
|
||||||
|
{ TODO: MOVE Rx,Ry; Bcc Rx,Rz,label; dealloc Rx ==> Bcc Ry,Rz,label }
|
||||||
|
end;
|
||||||
|
end;
|
||||||
|
|
||||||
A_LB,A_LBU,A_LH,A_LHU,A_LW,
|
A_LB,A_LBU,A_LH,A_LHU,A_LW,
|
||||||
A_ADD,A_ADDU,
|
A_ADD,A_ADDU,
|
||||||
A_ADDI,A_ADDIU,
|
A_ADDI,A_ADDIU,
|
||||||
|
Loading…
Reference in New Issue
Block a user