bslathi19/cc65
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Generalize the shared, non-removable reg load tests in RemoveRegLoads() to "instruction affects the other register"

Browse Source
This commit is contained in:
Alex Volkov
2026-03-29 17:05:20 -04:00
parent 321e47f0f3
commit de78048319
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/cc65/codeoptutil.c
View File

@@ -1252,19 +1252,19 @@ void RemoveRegLoads (StackOpData* D, LoadInfo* LI)
CHECK (!CanRemoveX || LI->X.LoadEntry->OPC != OP65_JSR || CHECK (!CanRemoveX || LI->X.LoadEntry->OPC != OP65_JSR ||
CE_IsCallTo (LI->X.LoadEntry, "ldaxysp")); CE_IsCallTo (LI->X.LoadEntry, "ldaxysp"));
/* When the A load is a runtime call and the X load cannot be removed, /* When the A load insn affects X reg (e.g. ldaxysp runtime), and the X
** we cannot remove the A load either. ** load cannot be removed, we cannot remove the A load either.
*/ */
if (CanRemoveA && LI->A.LoadEntry->OPC == OP65_JSR && !CanRemoveX) { if (CanRemoveA && (LI->A.LoadEntry->Chg & REG_X) != 0 && !CanRemoveX) {
/* A load is not removable */ /* A load is not removable */
LI->A.LoadEntry->Flags |= CEF_DONT_REMOVE; LI->A.LoadEntry->Flags |= CEF_DONT_REMOVE;
CanRemoveA = 0; CanRemoveA = 0;
} }
/* When the X load is a runtime call and the A load cannot be removed, /* When the X load insn affects A reg (e.g. ldaxysp runtime), and the A
** we cannot remove the X load either. ** load cannot be removed, we cannot remove the X load either.
*/ */
if (CanRemoveX && LI->X.LoadEntry->OPC == OP65_JSR && !CanRemoveA) { if (CanRemoveX && (LI->X.LoadEntry->Chg & REG_A) != 0 && !CanRemoveA) {
/* X load is not removable */ /* X load is not removable */
LI->X.LoadEntry->Flags |= CEF_DONT_REMOVE; LI->X.LoadEntry->Flags |= CEF_DONT_REMOVE;
CanRemoveX = 0; CanRemoveX = 0;