]> Shamusworld >> Repos - rmac/blobdiff - direct.c
Version bump for last commit. :-)
[rmac] / direct.c
index d6d6c3f2e63499d7301253b41f9881eb72daef08..c403b483128231b87b3184c288b2d12ccd0f70d5 100644 (file)
--- a/direct.c
+++ b/direct.c
@@ -1,7 +1,7 @@
 //
-// RMAC - Reboot's Macro Assembler for all Atari computers
+// RMAC - Renamed Macro Assembler for all Atari computers
 // DIRECT.C - Directive Handling
-// Copyright (C) 199x Landon Dyer, 2011-2020 Reboot and Friends
+// Copyright (C) 199x Landon Dyer, 2011-2021 Reboot and Friends
 // RMAC derived from MADMAC v1.07 Written by Landon Dyer, 1986
 // Source utilised with the kind permission of Landon Dyer
 //
 
 #define DEF_KW
 #include "kwtab.h"
-
+#define DEF_REG56
+#define DECL_REG56
+#include "56kregs.h"
+#define DEF_REG68
+#define DECL_REG68
+#include "68kregs.h"
+#define DEF_REGRISC
+#define DECL_REGRISC
+#include "riscregs.h"
 
 TOKEN exprbuf[128];                    // Expression buffer
 SYM * symbolPtr[1000000];      // Symbol pointers table
@@ -82,6 +90,7 @@ int d_prgflags(void);
 int d_opt(void);
 int d_dsp(void);
 int d_objproc(void);
+int d_align(void);
 void SetLargestAlignment(int);
 
 // Directive handler table
@@ -155,6 +164,7 @@ int (*dirtab[])() = {
        d_opt,                          // 66 .opt
        d_objproc,                      // 67 .objproc
        (void *)d_dsm,                  // 68 .dsm
+       d_align                         // 69 .align
 };
 
 
@@ -281,22 +291,22 @@ int d_org(void)
 
                switch (tok[0])
                {
-               case KW_X:
+               case REG56_X:
                        dsp_currentorg->memtype = ORG_X;
                        sectionToSwitch = M56001X;
                        break;
 
-               case KW_Y:
+               case REG56_Y:
                        dsp_currentorg->memtype = ORG_Y;
                        sectionToSwitch = M56001Y;
                        break;
 
-               case KW_P:
+               case REG56_P:
                        dsp_currentorg->memtype = ORG_P;
                        sectionToSwitch = M56001P;
                        break;
 
-               case KW_L:
+               case REG56_L:
                        dsp_currentorg->memtype = ORG_L;
                        sectionToSwitch = M56001L;
                        break;
@@ -396,15 +406,16 @@ int d_print(void)
                        formatting = 1;
 
                        // "X" & "L" get tokenized now... :-/ Probably should look into preventing this kind of thing from happening (was added with DSP56K code)
-                       if ((tok[1] != SYMBOL) && (tok[1] != KW_L) && (tok[1] != KW_X))
+                       // Note (ggn): This is now much less severe as it's localised for 56k only
+                       if ((tok[1] != SYMBOL) && (tok[1] != REG56_L) && (tok[1] != REG56_X))
                                goto token_err;
 
-                       if (tok[1] == KW_L)
+                       if (tok[1] == REG56_L)
                        {
                                wordlong = 1;
                                tok += 2;
                        }
-                       else if (tok[1] == KW_X)
+                       else if (tok[1] == REG56_X)
                        {
                                outtype = 0;
                                tok += 2;
@@ -536,7 +547,7 @@ int d_equrundef(void)
                {
                        // Reset the attributes of this symbol...
                        regname->sattr = 0;
-                       regname->sattre &= ~(EQUATEDREG | BANK_0 | BANK_1);
+                       regname->sattre &= ~EQUATEDREG;
                        regname->sattre |= UNDEF_EQUR;
                }
 
@@ -610,28 +621,45 @@ allright:
 
        tok += 2;
 
+       size = lseek(fd, 0L, SEEK_END);
+       pos = lseek(fd, 0L, SEEK_SET);
+
        if (*tok != EOL)
        {
-               // Check size parameter (can be omitted)
-               if (*tok++ == ',')
+               // Parse size and position parameters
+               uint64_t requested_size = -1;   // -1 means "not set" for these two
+
+               if (*tok++ != ',')
+               {
+                       close(fd);
+                       return error("expected comma after incbin filename");
+               }
+
+               if (*tok != EOL)
                {
                        if (*tok != ',')
                        {
-                               if (abs_expr(&size) != OK)
+                               if (abs_expr(&requested_size) != OK)
                                {
                                        close(fd);
                                        return ERROR;
                                }
+
+                               if ((int64_t)requested_size <= 0 || requested_size > size)
+                               {
+                                       close(fd);
+                                       return error("invalid incbin size requested");
+                               }
                        }
-                       else
-                               size = lseek(fd, 0L, SEEK_END);
-               }
 
-               // Check offset parameter (can be omitted)
-               if (*tok != EOL)
-               {
-                       if (*tok++ == ',')
+                       if (*tok != EOL)
                        {
+                               if (*tok++ != ',')
+                               {
+                                       close(fd);
+                                       return error("expected comma after size parameter");
+                               }
+
                                if (*tok != EOL)
                                {
                                        if (abs_expr(&pos) != OK)
@@ -640,26 +668,37 @@ allright:
                                                return ERROR;
                                        }
 
-                                       lseek(fd, pos, SEEK_SET);
-                                       size -= pos;
-                               }
-                               else
-                               {
-                                       // offset parameter omitted, so it's 0
-                                       pos = lseek(fd, 0L, SEEK_SET);
+                                       if ((int64_t)pos <= 0 || pos > size)
+                                       {
+                                               close(fd);
+                                               return error("invalid incbin position requested");
+                                       }
                                }
                        }
-                       else
-                               return error(comma_error);
+
+                       if (*tok != EOL)
+                       {
+                               close(fd);
+                               return error("extra characters following incbin");
+                       }
                }
-               else
-                       pos = lseek(fd, 0L, SEEK_SET);
-       }
-       else
-       {
-               // size & pos not given, so assume offset of 0 and all of the binary
-               size = lseek(fd, 0L, SEEK_END);
-               pos = lseek(fd, 0L, SEEK_SET);
+
+               // Adjust size if the user didn't specify it via the parameter
+               if (requested_size == -1)
+               {
+                       requested_size = size - pos;
+               }
+
+               // Are we going to read past the end of the file?
+               if (pos + requested_size > size)
+               {
+                       close(fd);
+                       return error("invalid combination of incbin position and size");
+               }
+               size = requested_size;
+
+               // All checks passed, let's seek to where the user requested, otherwise at file start
+               lseek(fd, pos, SEEK_SET);
        }
 
        chcheck(size);
@@ -694,16 +733,16 @@ allright:
 //
 int d_regbank0(void)
 {
-       // Set active register bank zero
-       regbank = BANK_0;
+       // Deprecated, it's not as if this did anything useful, ever
+       warn("regbank0 ignored");
        return 0;
 }
 
 
 int d_regbank1(void)
 {
-       // Set active register bank one
-       regbank = BANK_1;
+       // Deprecated, it's not as if this did anything useful, ever
+       warn("regbank1 ignored");
        return 0;
 }
 
@@ -826,6 +865,48 @@ int d_qphrase(void)
 }
 
 
+//
+// Adjust location to <alignment> bytes
+//
+int d_align(void)
+{
+       unsigned bytesToSkip;
+       uint64_t eval;
+
+       if (abs_expr(&eval) != OK)
+               return 0;
+
+       if (eval < 2)
+       {
+               return error("Invalid .align value specified");
+       }
+
+       if (dsp56001)
+       {
+               bytesToSkip = eval - sloc % eval;
+               D_ZEROFILL(bytesToSkip*3);
+               return 0;
+       }
+
+       bytesToSkip = eval - (rgpu || rdsp ? orgaddr : sloc) % eval;
+       if ( bytesToSkip != eval )
+       {
+               if ((scattr & SBSS) == 0)
+               {
+                       D_ZEROFILL(bytesToSkip);
+               }
+               else
+               {
+                       sloc += bytesToSkip;
+
+                       if (orgactive)
+                               orgaddr += bytesToSkip;
+               }
+       }
+       return 0;
+}
+
+
 //
 // Do auto-even.  This must be called ONLY if 'sloc' is odd.
 //
@@ -1130,6 +1211,7 @@ int d_ds(WORD siz)
        DEBUG { printf("Directive: .ds.[size] = %u, sloc = $%X\n", siz, sloc); }
 
        uint64_t eval;
+    WORD eattr;
 
        if ((cursect & (M6502 | M56KPXYL)) == 0)
        {
@@ -1137,8 +1219,8 @@ int d_ds(WORD siz)
                        auto_even();
        }
 
-       if (abs_expr(&eval) != OK)
-               return 0;
+       if (expr(exprbuf, &eval, &eattr, NULL) < 0)
+               return ERROR;
 
        // Check to see if the value being passed in is negative (who the hell does
        // that?--nobody does; it's the code gremlins, or rum, what does it)
@@ -1837,6 +1919,10 @@ int d_68000(void)
        SaveSection();
        SwitchSection(TEXT);
        activecpu = CPU_68000;
+       regbase = reg68base;    // Update register DFA tables
+       regtab = reg68tab;
+       regcheck = reg68check;
+       regaccept = reg68accept;
        return 0;
 }
 
@@ -1892,8 +1978,11 @@ int d_68060(void)
 //
 int d_68881(void)
 {
-       //d_68000();
        activefpu = FPU_68881;
+       regbase = reg68base;    // Update register DFA tables
+       regtab = reg68tab;
+       regcheck = reg68check;
+       regaccept = reg68accept;
        return 0;
 }
 
@@ -1903,8 +1992,11 @@ int d_68881(void)
 //
 int d_68882(void)
 {
-       //d_68000();
        activefpu = FPU_68882;
+       regbase = reg68base;    // Update register DFA tables
+       regtab = reg68tab;
+       regcheck = reg68check;
+       regaccept = reg68accept;
        return 0;
 }
 
@@ -1931,6 +2023,11 @@ int d_56001(void)
        if ((obj_format == LOD) || (obj_format == P56))
                SwitchSection(M56001P);
 
+       regbase = reg56base;    // Update register DFA tables
+       regtab = reg56tab;
+       regcheck = reg56check;
+       regaccept = reg56accept;
+       used_architectures |= M56001P | M56001X | M56001Y | M56001L;
        return 0;
 }
 
@@ -1957,7 +2054,12 @@ int d_gpu(void)
        rdsp = 0;                       // Unset DSP assembly
        robjproc = 0;           // Unset OP assembly
        dsp56001 = 0;           // Unset 56001 assembly
-       regbank = BANK_N;       // Set no default register bank
+
+       regbase = regriscbase;  // Update register DFA tables
+       regtab = regrisctab;
+       regcheck = regrisccheck;
+       regaccept = regriscaccept;
+       //used_architectures |= MGPU;   // TODO: Should GPU/DSP have their own dedicated sections in the long run?
        return 0;
 }
 
@@ -1984,7 +2086,12 @@ int d_dsp(void)
        rgpu = 0;                       // Unset GPU assembly
        robjproc = 0;           // Unset OP assembly
        dsp56001 = 0;           // Unset 56001 assembly
-       regbank = BANK_N;       // Set no default register bank
+
+       regbase = regriscbase;  // Update register DFA tables
+       regtab = regrisctab;
+       regcheck = regrisccheck;
+       regaccept = regriscaccept;
+       //used_architectures |= MDSP;   // TODO: Should GPU/DSP have their own dedicated sections in the long run?
        return 0;
 }
 
@@ -2063,7 +2170,7 @@ int d_cargs(void)
 
                        eval += 2;
                }
-               else if (*tok >= KW_D0 && *tok <= KW_A7)
+               else if (*tok >= REG68_D0 && *tok <= REG68_A7)
                {
                        if (reglist(&rlist) < 0)
                                return 0;
@@ -2078,13 +2185,13 @@ int d_cargs(void)
                {
                        switch ((int)*tok)
                        {
-                       case KW_USP:
-                       case KW_SSP:
-                       case KW_PC:
+                       case REG68_USP:
+                       case REG68_SSP:
+                       case REG68_PC:
                                eval += 2;
                                // FALLTHROUGH
-                       case KW_SR:
-                       case KW_CCR:
+                       case REG68_SR:
+                       case REG68_CCR:
                                eval += 2;
                                tok++;
                                break;
@@ -2195,7 +2302,7 @@ int d_cstruct(void)
 
                        tok++;
                }
-               else if (*tok >= KW_D0 && *tok <= KW_A7)
+               else if (*tok >= REG68_D0 && *tok <= REG68_A7)
                {
                        if (reglist(&rlist) < 0)
                                return 0;
@@ -2210,13 +2317,13 @@ int d_cstruct(void)
                {
                        switch ((int)*tok)
                        {
-                       case KW_USP:
-                       case KW_SSP:
-                       case KW_PC:
+                       case REG68_USP:
+                       case REG68_SSP:
+                       case REG68_PC:
                                eval += 2;
                                // FALLTHROUGH
-                       case KW_SR:
-                       case KW_CCR:
+                       case REG68_SR:
+                       case REG68_CCR:
                                eval += 2;
                                tok++;
                                break;
@@ -2257,6 +2364,7 @@ int d_objproc(void)
        rgpu = 0;                       // Unset GPU assembly
        rdsp = 0;                       // Unset DSP assembly
        dsp56001 = 0;           // Unset 56001 assembly
+       //used_architectures |= MOP;    // TODO: Should OP have its own dedicated section in the long run?
        return OK;
 }