]> git.sur5r.net Git - cc65/blobdiff - src/cc65/codeent.c
Fixed a bug
[cc65] / src / cc65 / codeent.c
index 61b41a479a556c255125340f40a15ccb8356c120..7cca0bd788863c11b41a2b60a53314736de64d02 100644 (file)
@@ -40,6 +40,7 @@
 #include "chartype.h"
 #include "check.h"
 #include "xmalloc.h"
+#include "xsprintf.h"
 
 /* cc65 */
 #include "codeinfo.h"
@@ -136,6 +137,8 @@ static int NumArg (const char* Arg, unsigned long* Num)
 static void SetUseChgInfo (CodeEntry* E, const OPCDesc* D)
 /* Set the Use and Chg in E */
 {
+    const ZPInfo* Info;
+
     /* If this is a subroutine call, or a jump to an external function,
      * lookup the information about this function and use it. The jump itself
      * does not change any registers, so we don't need to use the data from D.
@@ -145,10 +148,53 @@ static void SetUseChgInfo (CodeEntry* E, const OPCDesc* D)
        GetFuncInfo (E->Arg, &E->Use, &E->Chg);
     } else {
        /* Some other instruction. Use the values from the opcode description
-        * plus addressing mode info
+        * plus addressing mode info.
         */
        E->Use = D->Use | GetAMUseInfo (E->AM);
        E->Chg = D->Chg;
+
+       /* Check for special zero page registers used */
+       switch (E->AM) {
+
+           case AM65_ZP:
+           case AM65_ABS:
+           /* Be conservative: */
+           case AM65_ZPX:
+           case AM65_ABSX:
+           case AM65_ABSY:
+               Info = GetZPInfo (E->Arg);
+               if (Info && Info->ByteUse != REG_NONE) {
+                   if (E->OPC == OP65_ASL || E->OPC == OP65_DEC ||
+                       E->OPC == OP65_INC || E->OPC == OP65_LSR ||
+                       E->OPC == OP65_ROL || E->OPC == OP65_ROR ||
+                       E->OPC == OP65_TRB || E->OPC == OP65_TSB) {
+                       /* The zp loc is both, input and output */
+                       E->Chg |= Info->ByteUse;
+                       E->Use |= Info->ByteUse;
+                   } else if ((E->Info & OF_STORE) != 0) {
+                       /* Just output */
+                       E->Chg |= Info->ByteUse;
+                   } else {
+                       /* Input only */
+                       E->Use |= Info->ByteUse;
+                   }
+               }
+               break;
+
+           case AM65_ZPX_IND:
+           case AM65_ZP_INDY:
+           case AM65_ZP_IND:
+               Info = GetZPInfo (E->Arg);
+               if (Info && Info->ByteUse != REG_NONE) {
+                   /* These addressing modes will never change the zp loc */
+                   E->Use |= Info->WordUse;
+               }
+               break;
+
+           default:
+               /* Keep gcc silent */
+               break;
+       }
     }
 }
 
@@ -161,7 +207,7 @@ static void SetUseChgInfo (CodeEntry* E, const OPCDesc* D)
 
 
 CodeEntry* NewCodeEntry (opc_t OPC, am_t AM, const char* Arg,
-                        CodeLabel* JumpTo, LineInfo* LI)
+                        CodeLabel* JumpTo, LineInfo* LI)
 /* Create a new code entry, initialize and return it */
 {
     /* Get the opcode description */
@@ -179,6 +225,7 @@ CodeEntry* NewCodeEntry (opc_t OPC, am_t AM, const char* Arg,
     E->Size   = GetInsnSize (E->OPC, E->AM);
     E->JumpTo = JumpTo;
     E->LI     = UseLineInfo (LI);
+    E->RI     = 0;
     SetUseChgInfo (E, D);
     InitCollection (&E->Labels);
 
@@ -205,13 +252,16 @@ void FreeCodeEntry (CodeEntry* E)
     /* Release the line info */
     ReleaseLineInfo (E->LI);
 
+    /* Delete the register info */
+    CE_FreeRegInfo (E);
+
     /* Free the entry */
     xfree (E);
 }
 
 
 
-void ReplaceOPC (CodeEntry* E, opc_t OPC)
+void CE_ReplaceOPC (CodeEntry* E, opc_t OPC)
 /* Replace the opcode of the instruction. This will also replace related info,
  * Size, Use and Chg, but it will NOT update any arguments or labels.
  */
@@ -236,7 +286,7 @@ int CodeEntriesAreEqual (const CodeEntry* E1, const CodeEntry* E2)
 
 
 
-void AttachCodeLabel (CodeEntry* E, CodeLabel* L)
+void CE_AttachLabel (CodeEntry* E, CodeLabel* L)
 /* Attach the label to the entry */
 {
     /* Add it to the entries label list */
@@ -248,7 +298,7 @@ void AttachCodeLabel (CodeEntry* E, CodeLabel* L)
 
 
 
-void MoveCodeLabel (CodeLabel* L, CodeEntry* E)
+void CE_MoveLabel (CodeLabel* L, CodeEntry* E)
 /* Move the code label L from it's former owner to the code entry E. */
 {
     /* Delete the label from the owner */
@@ -261,19 +311,529 @@ void MoveCodeLabel (CodeLabel* L, CodeEntry* E)
 
 
 
-void CodeEntrySetArg (CodeEntry* E, const char* Arg)
-/* Set a new argument for the given code entry. An old string is deleted. */
+void CE_SetNumArg (CodeEntry* E, long Num)
+/* Set a new numeric argument for the given code entry that must already
+ * have a numeric argument.
+ */
 {
+    char Buf[16];
+
+    /* Check that the entry has a numerical argument */
+    CHECK (E->Flags & CEF_NUMARG);
+
+    /* Make the new argument string */
+    if (E->Size == 2) {
+       Num &= 0xFF;
+       xsprintf (Buf, sizeof (Buf), "$%02X", (unsigned) Num);
+    } else if (E->Size == 3) {
+       Num &= 0xFFFF;
+       xsprintf (Buf, sizeof (Buf), "$%04X", (unsigned) Num);
+    } else {
+       Internal ("Invalid instruction size in CE_SetNumArg");
+    }
+
     /* Free the old argument */
     FreeArg (E->Arg);
 
     /* Assign the new one */
-    E->Arg = GetArgCopy (Arg);
+    E->Arg = GetArgCopy (Buf);
+
+    /* Use the new numerical value */
+    E->Num = Num;
+}
+
+
+
+int CE_KnownImm (const CodeEntry* E)
+/* Return true if the argument of E is a known immediate value */
+{
+    return (E->AM == AM65_IMM && (E->Flags & CEF_NUMARG) != 0);
+}
+
+
+
+void CE_FreeRegInfo (CodeEntry* E)
+/* Free an existing register info struct */
+{
+    if (E->RI) {
+       FreeRegInfo (E->RI);
+       E->RI = 0;
+    }
 }
 
 
 
-void OutputCodeEntry (const CodeEntry* E, FILE* F)
+void CE_GenRegInfo (CodeEntry* E, RegContents* InputRegs)
+/* Generate register info for this instruction. If an old info exists, it is
+ * overwritten.
+ */
+{
+    /* Pointers to the register contents */
+    RegContents* In;
+    RegContents* Out;
+
+    /* Function register usage */
+    unsigned short Use, Chg;
+
+    /* If we don't have a register info struct, allocate one. */
+    if (E->RI == 0) {
+       E->RI = NewRegInfo (InputRegs);
+    } else {
+       if (InputRegs) {
+           E->RI->In  = *InputRegs;
+       } else {
+           RC_Invalidate (&E->RI->In);
+       }
+               E->RI->Out2 = E->RI->Out = E->RI->In;
+    }
+
+    /* Get pointers to the register contents */
+    In  = &E->RI->In;
+    Out        = &E->RI->Out;
+
+    /* Handle the different instructions */
+    switch (E->OPC) {
+
+       case OP65_ADC:
+           /* We don't know the value of the carry, so the result is
+            * always unknown.
+            */
+           Out->RegA = -1;
+           break;
+
+       case OP65_AND:
+                   if (In->RegA >= 0) {
+               if (CE_KnownImm (E)) {
+                   Out->RegA = In->RegA & (short) E->Num;
+               } else {
+                   Out->RegA = -1;
+               }
+           }
+           break;
+
+       case OP65_ASL:
+           if (E->AM == AM65_ACC && In->RegA >= 0) {
+               Out->RegA = (In->RegA << 1) & 0xFF;
+           } else if (E->AM == AM65_ZP) {
+               if ((E->Chg & REG_SREG_LO) != 0 && In->SRegLo >= 0) {
+                   Out->SRegLo = (In->SRegLo << 1) & 0xFF;
+               } else if ((E->Chg & REG_SREG_HI) != 0 && In->SRegHi >= 0) {
+                   Out->SRegHi = (In->SRegHi << 1) & 0xFF;
+               }
+           }
+           break;
+
+       case OP65_BCC:
+           break;
+
+       case OP65_BCS:
+           break;
+
+       case OP65_BEQ:
+           break;
+
+       case OP65_BIT:
+           break;
+
+       case OP65_BMI:
+           break;
+
+       case OP65_BNE:
+           break;
+
+       case OP65_BPL:
+           break;
+
+       case OP65_BRA:
+           break;
+
+       case OP65_BRK:
+           break;
+
+       case OP65_BVC:
+           break;
+
+       case OP65_BVS:
+           break;
+
+       case OP65_CLC:
+           break;
+
+       case OP65_CLD:
+           break;
+
+       case OP65_CLI:
+           break;
+
+       case OP65_CLV:
+           break;
+
+       case OP65_CMP:
+           break;
+
+       case OP65_CPX:
+           break;
+
+       case OP65_CPY:
+           break;
+
+       case OP65_DEA:
+           if (In->RegA >= 0) {
+               Out->RegA = (In->RegA - 1) & 0xFF;
+           }
+           break;
+
+       case OP65_DEC:
+           if (E->AM == AM65_ACC && In->RegA >= 0) {
+               Out->RegA = (In->RegA - 1) & 0xFF;
+           } else if (E->AM == AM65_ZP) {
+               if ((E->Chg & REG_SREG_LO) != 0 && In->SRegLo >= 0) {
+                   Out->SRegLo = (In->SRegLo - 1) & 0xFF;
+               } else if ((E->Chg & REG_SREG_HI) != 0 && In->SRegHi >= 0) {
+                   Out->SRegHi = (In->SRegHi - 1) & 0xFF;
+               }
+           }
+           break;
+
+       case OP65_DEX:
+                   if (In->RegX >= 0) {
+               Out->RegX = (In->RegX - 1) & 0xFF;
+           }
+           break;
+
+       case OP65_DEY:
+                   if (In->RegY >= 0) {
+               Out->RegY = (In->RegY - 1) & 0xFF;
+           }
+           break;
+
+       case OP65_EOR:
+                   if (In->RegA >= 0) {
+               if (CE_KnownImm (E)) {
+                   Out->RegA = In->RegA ^ (short) E->Num;
+               } else {
+                   Out->RegA = -1;
+               }
+           }
+           break;
+
+       case OP65_INA:
+           if (In->RegA >= 0) {
+               Out->RegA = (In->RegA + 1) & 0xFF;
+           }
+           break;
+
+       case OP65_INC:
+           if (E->AM == AM65_ACC && In->RegA >= 0) {
+               Out->RegA = (In->RegA + 1) & 0xFF;
+           } else if (E->AM == AM65_ZP) {
+               if ((E->Chg & REG_SREG_LO) != 0 && In->SRegLo >= 0) {
+                   Out->SRegLo = (In->SRegLo + 1) & 0xFF;
+               } else if ((E->Chg & REG_SREG_HI) != 0 && In->SRegHi >= 0) {
+                   Out->SRegHi = (In->SRegHi + 1) & 0xFF;
+               }
+           }
+           break;
+
+       case OP65_INX:
+           if (In->RegX >= 0) {
+               Out->RegX = (In->RegX + 1) & 0xFF;
+           }
+           break;
+
+       case OP65_INY:
+           if (In->RegY >= 0) {
+               Out->RegY = (In->RegY + 1) & 0xFF;
+           }
+           break;
+
+       case OP65_JCC:
+           break;
+
+       case OP65_JCS:
+           break;
+
+       case OP65_JEQ:
+           break;
+
+       case OP65_JMI:
+           break;
+
+       case OP65_JMP:
+           break;
+
+       case OP65_JNE:
+           break;
+
+       case OP65_JPL:
+           break;
+
+       case OP65_JSR:
+           /* Get the code info for the function */
+           GetFuncInfo (E->Arg, &Use, &Chg);
+           if (Chg & REG_A) {
+               Out->RegA = -1;
+           }
+           if (Chg & REG_X) {
+               Out->RegX = -1;
+           }
+           if (Chg & REG_Y) {
+               Out->RegY = -1;
+           }
+            if (Chg & REG_SREG_LO) {
+               Out->SRegLo = -1;
+           }
+           if (Chg & REG_SREG_HI) {
+               Out->SRegHi = -1;
+           }
+           break;
+
+       case OP65_JVC:
+           break;
+
+       case OP65_JVS:
+           break;
+
+       case OP65_LDA:
+           if (CE_KnownImm (E)) {
+               Out->RegA = (unsigned char) E->Num;
+           } else if (E->AM == AM65_ZP) {
+               if (E->Use & REG_SREG_LO) {
+                   Out->RegA = In->SRegLo;
+               } else if (E->Use & REG_SREG_HI) {
+                   Out->RegA = In->SRegHi;
+               } else {
+                   Out->RegA = -1;
+               }
+           } else {
+               /* A is now unknown */
+               Out->RegA = -1;
+           }
+           break;
+
+       case OP65_LDX:
+           if (CE_KnownImm (E)) {
+               Out->RegX = (unsigned char) E->Num;
+           } else if (E->AM == AM65_ZP) {
+               if (E->Use & REG_SREG_LO) {
+                   Out->RegX = In->SRegLo;
+               } else if (E->Use & REG_SREG_HI) {
+                   Out->RegX = In->SRegHi;
+               } else {
+                   Out->RegX = -1;
+               }
+           } else {
+               /* X is now unknown */
+               Out->RegX = -1;
+           }
+           break;
+
+       case OP65_LDY:
+           if (CE_KnownImm (E)) {
+               Out->RegY = (unsigned char) E->Num;
+           } else if (E->AM == AM65_ZP) {
+               if (E->Use & REG_SREG_LO) {
+                   Out->RegY = In->SRegLo;
+               } else if (E->Use & REG_SREG_HI) {
+                   Out->RegY = In->SRegHi;
+               } else {
+                   Out->RegY = -1;
+               }
+           } else {
+               /* Y is now unknown */
+               Out->RegY = -1;
+           }
+           break;
+
+       case OP65_LSR:
+           if (E->AM == AM65_ACC && In->RegA >= 0) {
+               Out->RegA = (In->RegA >> 1) & 0xFF;
+           } else if (E->AM == AM65_ZP) {
+               if ((E->Chg & REG_SREG_LO) != 0 && In->SRegLo >= 0) {
+                   Out->SRegLo = (In->SRegLo >> 1) & 0xFF;
+               } else if (E->Chg & REG_SREG_HI) {
+                   Out->SRegHi = (In->SRegHi >> 1) & 0xFF;
+               }
+           }
+           break;
+
+       case OP65_NOP:
+           break;
+
+       case OP65_ORA:
+           if (In->RegA >= 0) {
+               if (CE_KnownImm (E)) {
+                   Out->RegA = In->RegA | (short) E->Num;
+               } else {
+                   /* A is now unknown */
+                   Out->RegA = -1;
+               }
+           }
+           break;
+
+       case OP65_PHA:
+           break;
+
+       case OP65_PHP:
+           break;
+
+       case OP65_PHX:
+           break;
+
+       case OP65_PHY:
+           break;
+
+       case OP65_PLA:
+           Out->RegA = -1;
+           break;
+
+       case OP65_PLP:
+           break;
+
+       case OP65_PLX:
+           Out->RegX = -1;
+           break;
+
+       case OP65_PLY:
+           Out->RegY = -1;
+           break;
+
+       case OP65_ROL:
+           if (E->AM == AM65_ACC) {
+               Out->RegA = -1;
+           } else if (E->AM == AM65_ZP) {
+               if (E->Chg & REG_SREG_LO) {
+                   Out->SRegLo = -1;
+               } else if (E->Chg & REG_SREG_HI) {
+                   Out->SRegHi = -1;
+               }
+           }
+           break;
+
+       case OP65_ROR:
+           if (E->AM == AM65_ACC) {
+               Out->RegA = -1;
+           } else if (E->AM == AM65_ZP) {
+               if (E->Chg & REG_SREG_LO) {
+                   Out->SRegLo = -1;
+               } else if (E->Chg & REG_SREG_HI) {
+                   Out->SRegHi = -1;
+               }
+           }
+           break;
+
+       case OP65_RTI:
+           break;
+
+       case OP65_RTS:
+           break;
+
+       case OP65_SBC:
+           /* We don't know the value of the carry bit */
+           Out->RegA = -1;
+           break;
+
+       case OP65_SEC:
+           break;
+
+       case OP65_SED:
+           break;
+
+       case OP65_SEI:
+           break;
+
+       case OP65_STA:
+           if (E->AM == AM65_ZP) {
+               if (E->Chg & REG_SREG_LO) {
+                   Out->SRegLo = In->RegA;
+               } else if (E->Chg & REG_SREG_HI) {
+                   Out->SRegHi = In->RegA;
+               }
+           }
+           break;
+
+       case OP65_STX:
+           if (E->AM == AM65_ZP) {
+               if (E->Chg & REG_SREG_LO) {
+                   Out->SRegLo = In->RegX;
+               } else if (E->Chg & REG_SREG_HI) {
+                   Out->SRegHi = In->RegX;
+               }
+           }
+           break;
+
+       case OP65_STY:
+           if (E->AM == AM65_ZP) {
+               if (E->Chg & REG_SREG_LO) {
+                   Out->SRegLo = In->RegY;
+               } else if (E->Chg & REG_SREG_HI) {
+                   Out->SRegHi = In->RegY;
+               }
+           }
+           break;
+
+       case OP65_TAX:
+           Out->RegX = In->RegA;
+           break;
+
+       case OP65_TAY:
+           Out->RegY = In->RegA;
+           break;
+
+       case OP65_TRB:
+           /* For now... */
+           Out->RegA = -1;
+           break;
+
+       case OP65_TSB:
+           /* For now... */
+           Out->RegA = -1;
+           break;
+
+       case OP65_TSX:
+           Out->RegX = -1;
+           break;
+
+       case OP65_TXA:
+           Out->RegA = In->RegX;
+           break;
+
+       case OP65_TXS:
+           break;
+
+       case OP65_TYA:
+           Out->RegA = In->RegY;
+           break;
+
+       default:
+           break;
+
+    }
+}
+
+
+
+static char* RegInfoDesc (unsigned U, char* Buf)
+/* Return a string containing register info */
+{
+    Buf[0] = '\0';
+
+    strcat (Buf, U & REG_SREG_HI? "H" : "_");
+    strcat (Buf, U & REG_SREG_LO? "L" : "_");
+    strcat (Buf, U & REG_A?       "A" : "_");
+    strcat (Buf, U & REG_X?       "X" : "_");
+    strcat (Buf, U & REG_Y?       "Y" : "_");
+    strcat (Buf, U & REG_TMP1?    "T1" : "__");
+    strcat (Buf, U & REG_PTR1?    "1" : "_");
+    strcat (Buf, U & REG_PTR2?    "2" : "_");
+    strcat (Buf, U & REG_SAVE?    "V"  : "_");
+
+    return Buf;
+}
+
+
+
+void CE_Output (const CodeEntry* E, FILE* F)
 /* Output the code entry to a file */
 {
     const OPCDesc* D;
@@ -284,7 +844,7 @@ void OutputCodeEntry (const CodeEntry* E, FILE* F)
     unsigned LabelCount = CollCount (&E->Labels);
     unsigned I;
     for (I = 0; I < LabelCount; ++I) {
-       OutputCodeLabel (CollConstAt (&E->Labels, I), F);
+       CL_Output (CollConstAt (&E->Labels, I), F);
     }
 
     /* Get the opcode description */
@@ -296,6 +856,7 @@ void OutputCodeEntry (const CodeEntry* E, FILE* F)
     /* Print the operand */
     switch (E->AM) {
 
+       case AM_IMP:
        case AM65_IMP:
            /* implicit */
            break;
@@ -305,16 +866,18 @@ void OutputCodeEntry (const CodeEntry* E, FILE* F)
            Chars += fprintf (F, "%*sa", 9-Chars, "");
            break;
 
-       case AM65_IMM:
+       case AM_IMM:
+       case AM65_IMM:
            /* immidiate */
            Chars += fprintf (F, "%*s#%s", 9-Chars, "", E->Arg);
            break;
 
+       case AM_ABS:
        case AM65_ZP:
        case AM65_ABS:
            /* zeropage and absolute */
            Chars += fprintf (F, "%*s%s", 9-Chars, "", E->Arg);
-           break;
+                   break;
 
        case AM65_ZPX:
        case AM65_ABSX:
@@ -355,16 +918,14 @@ void OutputCodeEntry (const CodeEntry* E, FILE* F)
 
     /* Print usage info if requested by the debugging flag */
     if (Debug) {
-       fprintf (F,
-                "%*s; USE: %c%c%c CHG: %c%c%c SIZE: %u\n",
-                30-Chars, "",
-                (E->Use & REG_A)? 'A' : '_',
-                (E->Use & REG_X)? 'X' : '_',
-                (E->Use & REG_Y)? 'Y' : '_',
-                (E->Chg & REG_A)? 'A' : '_',
-                (E->Chg & REG_X)? 'X' : '_',
-                (E->Chg & REG_Y)? 'Y' : '_',
-                E->Size);
+       char Use [128];
+       char Chg [128];
+               fprintf (F,
+                        "%*s; USE: %-20s CHG: %-20s SIZE: %u\n",
+                        30-Chars, "",
+                RegInfoDesc (E->Use, Use),
+                RegInfoDesc (E->Chg, Chg),
+                E->Size);
     } else {
        /* Terminate the line */
        fprintf (F, "\n");
@@ -375,3 +936,4 @@ void OutputCodeEntry (const CodeEntry* E, FILE* F)
 
 
 
+