]> Shamusworld >> Repos - rmac/blobdiff - direct.c
Actually implement ^^FILESIZE this time :)
[rmac] / direct.c
index 32e5dd19233c679bab7845f0f6a41f51bc030452..b3f51f3c3b73356c638a0dc9b087cc20f67ab58d 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-2019 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
 //
@@ -51,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);
@@ -153,6 +154,7 @@ int (*dirtab[])() = {
        d_nofpu,                        // 65 nofpu
        d_opt,                          // 66 .opt
        d_objproc,                      // 67 .objproc
+       d_dsm,                          // 68 .dsm
 };
 
 
@@ -1074,11 +1076,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
@@ -1094,18 +1096,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);
        }
 
        ErrorIfNotAtEOL();
-       return 0;
+       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)
 {
@@ -1191,28 +1261,7 @@ int d_dc(WORD siz)
                                        if (eattr & FLOAT)
                                        {
                                                double fval = *(double *)&eval;
-
-                                               if (fval >= 1)
-                                               {
-                                                       warn("value clamped to +1.");
-                                                       eval = 0x7fffff;
-                                               }
-                                               else if (fval <= -1)
-                                               {
-                                                       warn("value clamped to -1.");
-                                                       eval = 0x800000;
-                                               }
-                                               else
-                                               {
-                                                       // Convert fraction to 24 bits fixed point with sign and rounding
-                                                       // Yeah, that cast to int32_t has to be there because casting
-                                                       // a float to unsigned int is "undefined" according to the C
-                                                       // standard. Which most compilers seem to do the sensible thing
-                                                       // and just cast the f**king value properly, except gcc 4.x.x
-                                                       // for arm (tested on raspbian).
-                                                       // Thanks, C and gcc! Thanks for making me waste a few hours \o/
-                                                       eval = 0;//!!! FIX !!! (uint32_t)(int32_t)round(fval*(1 << 23));
-                                               }
+                                               eval = DoubleToDSPFloat(fval);
                                        }
                                        else
                                        {
@@ -1227,10 +1276,11 @@ int d_dc(WORD siz)
                        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.
+                               // 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:
@@ -1245,27 +1295,7 @@ l_parse_loop:
                                        if (eattr & FLOAT)
                                        {
                                                float fval = *(float *)&eval;
-                                               if (fval >= 1)
-                                               {
-                                                       warn("value clamped to +1.");
-                                                       eval = 0x7fffff;
-                                               }
-                                               else if (fval <= -1)
-                                               {
-                                                       warn("value clamped to -1.");
-                                                       eval = 0x800000;
-                                               }
-                                               else
-                                               {
-                                                       // Convert fraction to 24 bits fixed point with sign and rounding
-                                                       // Yeah, that cast to int32_t has to be there because casting
-                                                       // a float to unsigned int is "undefined" according to the C
-                                                       // standard. Which most compilers seem to do the sensible thing
-                                                       // and just cast the f**king value properly, except gcc 4.x.x
-                                                       // for arm (tested on raspbian).
-                                                       // Thanks, C and gcc! Thanks for making me waste a few hours \o/
-                                                       eval = 0;//!!! FIX !!! (uint32_t)(int32_t)round(fval*(1 << 23));
-                                               }
+                                               eval = DoubleToDSPFloat(fval);
                                        }
                                        else
                                        {
@@ -1305,6 +1335,7 @@ l_parse_loop:
                                }
 
                        }
+
                        goto comma;
                }
 
@@ -1328,6 +1359,7 @@ l_parse_loop:
                        }
 
                        break;
+
                case SIZW:
                case SIZN:
                        if (!defined)
@@ -1351,6 +1383,7 @@ l_parse_loop:
                        }
 
                        break;
+
                case SIZL:
                        // Shamus: Why can't we do longs in 6502 mode?
                        if (m6502)
@@ -1358,11 +1391,7 @@ l_parse_loop:
 
                        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
@@ -1377,34 +1406,33 @@ l_parse_loop:
                        }
 
                        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
                        {
@@ -1419,6 +1447,7 @@ l_parse_loop:
                        }
 
                        break;
+
                case SIZD:
                        // 64-bit double size
                        if (m6502)
@@ -1426,8 +1455,9 @@ l_parse_loop:
 
                        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
                        {
@@ -1442,6 +1472,7 @@ l_parse_loop:
                        }
 
                        break;
+
                case SIZX:
                        if (m6502)
                                return error(in_6502mode);
@@ -1451,8 +1482,9 @@ l_parse_loop:
 
                        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
                        {
@@ -1584,11 +1616,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--)
        {