paweld/fpc
1
0
Fork 0
mirror of https://gitlab.com/freepascal.org/fpc/source.git synced 2025-04-18 05:00:07 +02:00
Code Issues Packages Projects Releases Wiki Activity

Avoidd to use hp1 after calling RemoveInstruction, as the VMT is invalidated

Browse Source
This commit is contained in:
Pierre Muller 2022-11-11 22:20:10 +00:00
parent cfc77315ca
commit 71df6b83c6
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
compiler/x86/aoptx86.pas
View File

@ -5942,13 +5942,13 @@ unit aoptx86;
{ Overflow; abort }
Exit;
RemoveInstruction(hp1);
if (ThisConst = 0) then
begin
DebugMsg(SPeepholeOptimization + 'Arithmetic combine: ' +
debug_op2str(taicpu(hp1).opcode) + ' $' + debug_tostr(taicpu(hp1).oper[0]^.val) + ',' + debug_operstr(taicpu(hp1).oper[1]^) + '; ' +
debug_op2str(taicpu(p).opcode) + ' $' + debug_tostr(taicpu(p).oper[0]^.val) + ',' + debug_operstr(taicpu(p).oper[1]^) + ' cancel out (NOP)', p);
RemoveInstruction(hp1);
hp1 := tai(p.next);
RemoveInstruction(p); { Note, the choice to not use RemoveCurrentp is deliberate }
if not GetLastInstruction(hp1, p) then
@ -5967,6 +5967,7 @@ unit aoptx86;
debug_op2str(taicpu(p).opcode) + ' $' + debug_tostr(taicpu(p).oper[0]^.val) + ',' + debug_operstr(taicpu(p).oper[1]^) + ' -> ' +
debug_op2str(taicpu(p).opcode) + ' $' + debug_tostr(ThisConst) + ' ' + debug_operstr(taicpu(p).oper[1]^), p);
RemoveInstruction(hp1);
taicpu(p).loadconst(0, ThisConst);
end;