]> Shamusworld >> Repos - rmac/blobdiff - direct.c
Fix a small buglet in the last patch. :-)
[rmac] / direct.c
index 294ea4ac32649b19846bc8a69f5951c5ca522278..53d7c6ae48aeab8126bc826e24abda4edaf3c3e6 100644 (file)
--- a/direct.c
+++ b/direct.c
@@ -1,7 +1,7 @@
 //
 // RMAC - Reboot's Macro Assembler for all Atari computers
 // DIRECT.C - Directive Handling
-// Copyright (C) 199x Landon Dyer, 2011-2018 Reboot and Friends
+// Copyright (C) 199x Landon Dyer, 2011-2020 Reboot and Friends
 // RMAC derived from MADMAC v1.07 Written by Landon Dyer, 1986
 // Source utilised with the kind permission of Landon Dyer
 //
@@ -9,6 +9,7 @@
 #include "direct.h"
 #include "6502.h"
 #include "amode.h"
+#include "dsp56k.h"
 #include "error.h"
 #include "expr.h"
 #include "fltpoint.h"
@@ -50,6 +51,7 @@ int d_abs(void);
 int d_comm(void);
 int d_dc(WORD);
 int d_ds(WORD);
+int d_dsm(WORD);
 int d_dcb(WORD);
 int d_globl(void);
 int d_gpu(void);
@@ -152,6 +154,7 @@ int (*dirtab[])() = {
        d_nofpu,                        // 65 nofpu
        d_opt,                          // 66 .opt
        d_objproc,                      // 67 .objproc
+       (void *)d_dsm,                  // 68 .dsm
 };
 
 
@@ -218,11 +221,13 @@ int d_org(void)
 {
        uint64_t address;
 
-       if (!rgpu && !rdsp && !robjproc && !m6502 && !dsp56001)
-               return error(".org permitted only in GPU/DSP/OP, 56001 and 6502 sections");
+       if (!rgpu && !rdsp && !robjproc && !m6502 && !dsp56001 && !(obj_format == RAW))
+               return error(".org permitted only in GPU/DSP/OP, 56001, 6502 and 68k (with -fr switch) sections");
 
        // M56K can leave the expression off the org for some reason :-/
-       if ((abs_expr(&address) == ERROR) && !dsp56001)
+       // (It's because the expression is non-standard, and so we have to look at
+       // it in isolation)
+       if (!dsp56001 && (abs_expr(&address) == ERROR))
        {
                error("cannot determine org'd address");
                return ERROR;
@@ -255,9 +260,105 @@ int d_org(void)
        }
        else if (dsp56001)
        {
+               // Only mark segments we actually wrote something
+               if (chptr != dsp_currentorg->start && dsp_written_data_in_current_org)
+               {
+                       dsp_currentorg->end = chptr;
+                       dsp_currentorg++;
+               }
+
+               // Maybe we switched from a non-DSP section (TEXT, DATA, etc) and
+               // scode isn't initialised yet. Not that it's going to be a valid
+               // scenario, but if we try it anyhow it's going to lead to a crash. So
+               // let's fudge a value of 0 and get on with it.
+               orgaddr = (scode != NULL ? sloc : 0);
+               SaveSection();
+
+               if (tok[1] != ':')
+                       return error(syntax_error);
+
+               int sectionToSwitch = 0;
+
+               switch (tok[0])
+               {
+               case KW_X:
+                       dsp_currentorg->memtype = ORG_X;
+                       sectionToSwitch = M56001X;
+                       break;
+
+               case KW_Y:
+                       dsp_currentorg->memtype = ORG_Y;
+                       sectionToSwitch = M56001Y;
+                       break;
+
+               case KW_P:
+                       dsp_currentorg->memtype = ORG_P;
+                       sectionToSwitch = M56001P;
+                       break;
+
+               case KW_L:
+                       dsp_currentorg->memtype = ORG_L;
+                       sectionToSwitch = M56001L;
+                       break;
+
+               default:
+                       return error("unknown type in ORG");
+               }
+
+               if ((obj_format == LOD) || (obj_format == P56))
+                       SwitchSection(sectionToSwitch);
+
+               tok += 2;
+               chcheck(3); // Ensure we got a valid address to write
+               dsp_currentorg->chunk = scode;  // Mark down which chunk this org starts from (will be needed when outputting)
+
+               if (*tok == EOL)
+               {
+                       // Well, the user didn't specify an address at all so we'll have to
+                       // use the last used address of that section (or 0 if there wasn't one)
+                       address = orgaddr;
+                       dsp_currentorg->start = chptr;
+                       dsp_currentorg->orgadr = orgaddr;
+               }
+               else
+               {
+                       if (abs_expr(&address) == ERROR)
+                       {
+                               error("cannot determine org'd address");
+                               return ERROR;
+                       }
+
+                       dsp_currentorg->start = chptr;
+                       dsp_currentorg->orgadr = (uint32_t)address;
+                       sect[cursect].orgaddr = (uint32_t)address;
+               }
+
+               if (address > DSP_MAX_RAM)
+               {
+                       return error(range_error);
+               }
+
+               dsp_written_data_in_current_org = 0;
+
+               // Copied from 6502 above: kludge `lsloc' so the listing generator
+               // doesn't try to spew out megabytes.
+               lsloc = sloc = (int32_t)address;
+// N.B.: It seems that by enabling this, even though it works elsewhere, will cause symbols to royally fuck up.  Will have to do some digging to figure out why.
+//             orgactive = 1;
        }
+       else
+       {
+               // If we get here we assume it's 68k with RAW output, so this is allowed
+               if (orgactive)
+               {
+                       return error("In 68k mode only one .org statement is allowed");
+               }
 
-       at_eol();
+               org68k_address = address;
+               org68k_active = 1;
+       }
+
+       ErrorIfNotAtEOL();
        return 0;
 }
 
@@ -280,7 +381,7 @@ int d_print(void)
 
        while (*tok != EOL)
        {
-               switch(*tok)
+               switch (*tok)
                {
                case STRING:
                        sprintf(prntstr, "%s", string[tok[1]]);
@@ -312,7 +413,7 @@ int d_print(void)
                        {
                                strcpy(prntstr, string[tok[2]]);
 
-                               switch(prntstr[0])
+                               switch (prntstr[0])
                                {
                                case 'l': case 'L': wordlong = 1; break;
                                case 'w': case 'W': wordlong = 0; break;
@@ -458,13 +559,13 @@ int d_noclear(void)
 
 
 //
-// Include binary file
+// Include binary file (can add addition size & position params, comma separated)
 //
 int d_incbin(void)
 {
        int fd;
        int bytes = 0;
-       long pos, size, bytesRead;
+       uint64_t pos, size, bytesRead;
        char buf1[256];
        int i;
 
@@ -484,7 +585,9 @@ int d_incbin(void)
        // Attempt to open the include file in the current directory, then (if that
        // failed) try list of include files passed in the enviroment string or by
        // the "-d" option.
-       if ((fd = open(string[tok[1]], _OPEN_INC)) < 0)
+       TOKEN filename = tok[1];
+
+       if ((fd = open(string[filename], _OPEN_INC)) < 0)
        {
                for(i=0; nthpath("RMACPATH", i, buf1)!=0; i++)
                {
@@ -494,29 +597,81 @@ int d_incbin(void)
                        if (fd > 0 && buf1[fd - 1] != SLASHCHAR)
                                strcat(buf1, SLASHSTRING);
 
-                       strcat(buf1, string[tok[1]]);
+                       strcat(buf1, string[filename]);
 
                        if ((fd = open(buf1, _OPEN_INC)) >= 0)
                                goto allright;
                }
 
-               return error("cannot open: \"%s\"", string[tok[1]]);
+               return error("cannot open: \"%s\"", string[filename]);
        }
 
 allright:
 
-       size = lseek(fd, 0L, SEEK_END);
-       pos = lseek(fd, 0L, SEEK_SET);
+       tok += 2;
+
+       if (*tok != EOL)
+       {
+               // Check size parameter (can be omitted)
+               if (*tok++ == ',')
+               {
+                       if (*tok != ',')
+                       {
+                               if (abs_expr(&size) != OK)
+                               {
+                                       close(fd);
+                                       return ERROR;
+                               }
+                       }
+                       else
+                               size = lseek(fd, 0L, SEEK_END);
+               }
+
+               // Check offset parameter (can be omitted)
+               if (*tok != EOL)
+               {
+                       if (*tok++ == ',')
+                       {
+                               if (*tok != EOL)
+                               {
+                                       if (abs_expr(&pos) != OK)
+                                       {
+                                               close(fd);
+                                               return ERROR;
+                                       }
+
+                                       lseek(fd, pos, SEEK_SET);
+                                       size -= pos;
+                               }
+                               else
+                               {
+                                       // offset parameter omitted, so it's 0
+                                       pos = lseek(fd, 0L, SEEK_SET);
+                               }
+                       }
+                       else
+                               return error(comma_error);
+               }
+               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);
+       }
+
        chcheck(size);
 
-       DEBUG { printf("INCBIN: File '%s' is %li bytes.\n", string[tok[1]], size); }
+       DEBUG { printf("INCBIN: File '%s' is %li bytes.\n", string[filename], size); }
 
        char * fileBuffer = (char *)malloc(size);
        bytesRead = read(fd, fileBuffer, size);
 
        if (bytesRead != size)
        {
-               error("was only able to read %li bytes from binary file (%s, %li bytes)", bytesRead, string[tok[1]], size);
+               error("was only able to read %li bytes from binary file (%s, %li bytes)", bytesRead, string[filename], size);
                return ERROR;
        }
 
@@ -830,7 +985,7 @@ int d_assert(void)
                        break;
        }
 
-       at_eol();
+       ErrorIfNotAtEOL();
        return 0;
 }
 
@@ -986,11 +1141,11 @@ int d_ds(WORD siz)
                return 0;
 
        // 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, that does it)
-       // N.B.: Since 'eval' is of type uint32_t, if it goes negative, it will have
-       //       its high bit set.
-       if (eval & 0x80000000)
-               return error("negative sizes not allowed");
+       // that?--nobody does; it's the code gremlins, or rum, what does it)
+       // N.B.: Since 'eval' is of type uint64_t, if it goes negative, it will
+       //       have its high bit set.
+       if (eval & 0x8000000000000000)
+               return error("negative sizes not allowed in DS");
 
        // In non-TDB section (BSS, ABS and M6502) just advance the location
        // counter appropriately. In TDB sections, deposit (possibly large) chunks
@@ -1006,18 +1161,86 @@ int d_ds(WORD siz)
 
                just_bss = 1;                                   // No data deposited (8-bit CPU mode)
        }
+       else if (cursect & M56KPXYL)
+       {
+               // Change segment instead of marking blanks.
+               // Only mark segments we actually wrote something
+               if (chptr != dsp_currentorg->start && dsp_written_data_in_current_org)
+               {
+                       dsp_currentorg->end = chptr;
+                       dsp_currentorg++;
+                       dsp_currentorg->memtype = dsp_currentorg[-1].memtype;
+               }
+
+               listvalue((uint32_t)eval);
+               sloc += (uint32_t)eval;
+
+               // And now let's create a new segment
+               dsp_currentorg->start = chptr;
+               dsp_currentorg->chunk = scode;  // Mark down which chunk this org starts from (will be needed when outputting)
+               sect[cursect].orgaddr = sloc;
+               dsp_currentorg->orgadr = sloc;
+               dsp_written_data_in_current_org = 0;
+
+               just_bss = 1;                                   // No data deposited
+       }
        else
        {
-               dep_block(eval, siz, 0, (WORD)(DEFINED | ABS), NULL);
+               dep_block(eval, siz, 0, (DEFINED | ABS), NULL);
        }
 
-       at_eol();
-       return 0;
+       ErrorIfNotAtEOL();
+       return OK;
 }
 
 
 //
-// dc.b, dc.w / dc, dc.l, dc.i, dc.q, dc.d
+// dsm[.siz] expression
+// Define modulo storage
+// Quoting the Motorola assembler manual:
+// "The DSM directive reserves a block of memory the length of which in words is equal to
+// the value of <expression>.If the runtime location counter is not zero, this directive first
+// advances the runtime location counter to a base address that is a multiple of 2k, where
+// 2k >= <expression>."
+// The kicker of course is written a few sentences after:
+// "<label>, if present, will be assigned the value of the runtime location counter after a valid
+// base address has been established."
+//
+int d_dsm(WORD siz)
+{
+       TOKEN * tok_current = tok;  // Keep track of where tok was when we entered this procedure
+       uint64_t eval;
+
+       if (abs_expr(&eval) != OK)
+               return 0;
+
+       // Round up to the next highest power of 2
+       // Nicked from https://graphics.stanford.edu/~seander/bithacks.html#RoundUpPowerOf2
+       eval--;
+       eval |= eval >> 1;
+       eval |= eval >> 2;
+       eval |= eval >> 4;
+       eval |= eval >> 8;
+       eval |= eval >> 16;
+
+       int units_to_skip;
+       units_to_skip = eval + 1 - sloc;
+       sloc += units_to_skip;          // Bump up sloc - TODO: check if this goes over the RAM limits?
+
+       // If a label has been defined in the same line as dsm, its value also needs to be adjusted
+       if (label_defined)
+       {
+               SYM * label = lookup(label_defined, LABEL, 0);
+               label->svalue += units_to_skip;
+       }
+
+       tok = tok_current;              // Rewind tok back to where it was
+       return d_ds(siz);               // And let d_ds take over from here
+}
+
+
+//
+// dc.b, dc.w / dc, dc.l, dc.i, dc.q, dc.d, dc.s, dc.x
 //
 int d_dc(WORD siz)
 {
@@ -1029,7 +1252,9 @@ int d_dc(WORD siz)
                return error("illegal initialization of section");
 
        // Do an auto_even if it's not BYTE sized (hmm, should we be doing this???)
-       if (cursect != M6502 && (siz != SIZB) && (sloc & 1))
+       if ((cursect != M6502) && (cursect != M56001P) && (cursect != M56001X)
+               && (cursect != M56001Y) && (cursect != M56001L)
+               && (siz != SIZB) && (sloc & 1))
                auto_even();
 
        // Check to see if we're trying to set LONGS on a non 32-bit aligned
@@ -1086,6 +1311,99 @@ int d_dc(WORD siz)
                uint16_t tdb = eattr & TDB;
                uint16_t defined = eattr & DEFINED;
 
+// N.B.: This is awful.  This needs better handling, rather than just bodging something in that, while works, is basically an ugly wart on the assembler.  !!! FIX !!!
+               if (dsp56001)
+               {
+                       if (cursect != M56001L)
+                       {
+                               if (!defined)
+                               {
+                                       AddFixup(FU_DSPIMM24 | FU_SEXT, sloc, exprbuf);
+                                       D_dsp(0);
+                               }
+                               else
+                               {
+                                       if (eattr & FLOAT)
+                                       {
+                                               double fval = *(double *)&eval;
+                                               eval = DoubleToDSPFloat(fval);
+                                       }
+                                       else
+                                       {
+                                               if ((uint32_t)eval + 0x1000000 >= 0x2000000)
+                                                       return error(range_error);
+                                       }
+
+                                       // Deposit DSP word (24-bit)
+                                       D_dsp(eval);
+                               }
+                       }
+                       else
+                       {
+                               // In L: we deposit stuff to both X: and Y: instead
+                               // We will be a bit lazy and require that there is a 2nd value
+                               // in the same source line. (Motorola's assembler can parse
+                               // 12-digit hex values, which we can't do at the moment) This
+                               // of course requires to parse 2 values in one pass. If there
+                               // isn't another value in this line, assume X: value is 0.
+                               int secondword = 0;
+                               uint32_t evaly;
+l_parse_loop:
+
+                               if (!defined)
+                               {
+                                       AddFixup(FU_DSPIMM24 | FU_SEXT, sloc, exprbuf);
+                                       D_dsp(0);
+                               }
+                               else
+                               {
+                                       if (eattr & FLOAT)
+                                       {
+                                               float fval = *(float *)&eval;
+                                               eval = DoubleToDSPFloat(fval);
+                                       }
+                                       else
+                                       {
+                                               if (eval + 0x1000000 >= 0x2000000)
+                                                       return error(range_error);
+                                       }
+
+                                       // Parse 2nd value if we didn't do this yet
+                                       if (secondword == 0)
+                                       {
+                                               evaly = (uint32_t)eval;
+                                               secondword = 1;
+
+                                               if (*tok != ':')
+                                               {
+                                                       // If we don't have a : then we're probably at EOL,
+                                                       // which means the X: value will be 0
+                                                       eval = 0;
+                                                       ErrorIfNotAtEOL();
+                                               }
+                                               else
+                                               {
+                                                       tok++; // Eat the comma;
+
+                                                       if (expr(exprbuf, &eval, &eattr, NULL) != OK)
+                                                               return 0;
+
+                                                       defined = (WORD)(eattr & DEFINED);
+                                                       goto l_parse_loop;
+                                               }
+                                       }
+
+                                       // Deposit DSP words (24-bit)
+                                       D_dsp(eval);
+                                       D_dsp(evaly);
+                                       sloc--; // We do write 2 DSP words but as far as L: space is concerned we actually advance our counter by one
+                               }
+
+                       }
+
+                       goto comma;
+               }
+
                switch (siz)
                {
                case SIZB:
@@ -1106,6 +1424,7 @@ int d_dc(WORD siz)
                        }
 
                        break;
+
                case SIZW:
                case SIZN:
                        if (!defined)
@@ -1129,6 +1448,7 @@ int d_dc(WORD siz)
                        }
 
                        break;
+
                case SIZL:
                        // Shamus: Why can't we do longs in 6502 mode?
                        if (m6502)
@@ -1136,11 +1456,7 @@ int d_dc(WORD siz)
 
                        if (!defined)
                        {
-                               if (movei)
-                                       AddFixup(FU_LONG | FU_MOVEI, sloc, exprbuf);
-                               else
-                                       AddFixup(FU_LONG, sloc, exprbuf);
-
+                               AddFixup(FU_LONG | (movei ? FU_MOVEI : 0), sloc, exprbuf);
                                D_long(0);
                        }
                        else
@@ -1155,34 +1471,33 @@ int d_dc(WORD siz)
                        }
 
                        break;
+
                case SIZQ:
                        // 64-bit size
                        if (m6502)
                                return error(in_6502mode);
 
-                       // Shamus: We only handle DC.Q type stuff, will have to add fixups
-                       //         and stuff later (maybe... might not be needed...)
                        // DEFINITELY NEED FIXUPS HERE!
                        if (!defined)
                        {
                                AddFixup(FU_QUAD, sloc, exprbuf);
-                               D_quad(0LL);
-                       }
-                       else
-                       {
-                               D_quad(eval);
+                               eval = 0;
                        }
 
+                       D_quad(eval);
                        break;
+
                case SIZS:
                        // 32-bit float size
                        if (m6502)
                                return error(in_6502mode);
 
+/* Seems to me that if something is undefined here, then that should be an error.  Likewise for the D & X variants. */
                        if (!defined)
                        {
-                               AddFixup(FU_FLOATSING, sloc, exprbuf);
-                               D_long(0);
+//                             AddFixup(FU_FLOATSING, sloc, exprbuf);
+//                             D_long(0);
+                               return error("labels not allowed in floating point expressions");
                        }
                        else
                        {
@@ -1197,6 +1512,7 @@ int d_dc(WORD siz)
                        }
 
                        break;
+
                case SIZD:
                        // 64-bit double size
                        if (m6502)
@@ -1204,8 +1520,9 @@ int d_dc(WORD siz)
 
                        if (!defined)
                        {
-                               AddFixup(FU_FLOATDOUB, sloc, exprbuf);
-                               D_quad(0LL);
+//                             AddFixup(FU_FLOATDOUB, sloc, exprbuf);
+//                             D_quad(0LL);
+                               return error("labels not allowed in floating point expressions");
                        }
                        else
                        {
@@ -1220,6 +1537,7 @@ int d_dc(WORD siz)
                        }
 
                        break;
+
                case SIZX:
                        if (m6502)
                                return error(in_6502mode);
@@ -1229,8 +1547,9 @@ int d_dc(WORD siz)
 
                        if (!defined)
                        {
-                               AddFixup(FU_FLOATEXT, sloc, exprbuf);
-                               D_extend(extDbl);
+//                             AddFixup(FU_FLOATEXT, sloc, exprbuf);
+//                             D_extend(extDbl);
+                               return error("labels not allowed in floating point expressions");
                        }
                        else
                        {
@@ -1252,7 +1571,7 @@ comma:
                        break;
        }
 
-       at_eol();
+       ErrorIfNotAtEOL();
        return 0;
 }
 
@@ -1362,11 +1681,8 @@ int d_init(WORD def_siz)
 //
 int dep_block(uint32_t count, WORD siz, uint32_t eval, WORD eattr, TOKEN * exprbuf)
 {
-       WORD tdb;
-       WORD defined;
-
-       tdb = (WORD)(eattr & TDB);
-       defined = (WORD)(eattr & DEFINED);
+       WORD tdb = eattr & TDB;
+       WORD defined = eattr & DEFINED;
 
        while (count--)
        {
@@ -1480,7 +1796,7 @@ int d_comm(void)
                return 0;
 
        sym->svalue = eval;                                     // Install common symbol's size
-       at_eol();
+       ErrorIfNotAtEOL();
        return 0;
 }
 
@@ -1612,7 +1928,7 @@ int d_56001(void)
        rgpu = rdsp = robjproc = 0;
        SaveSection();
 
-       if (obj_format == LOD || obj_format == P56)
+       if ((obj_format == LOD) || (obj_format == P56))
                SwitchSection(M56001P);
 
        return 0;