]> git.sur5r.net Git - cc65/blobdiff - src/cc65/compile.c
Disallow global variable declarations with an initializer.
[cc65] / src / cc65 / compile.c
index 411c31357f60b3a494e2ce053599aa3b70e97f6a..f01ddf6c580b9463735d3f145a964b20fb72070c 100644 (file)
@@ -1,15 +1,15 @@
 /*****************************************************************************/
 /*                                                                           */
-/*                                compile.c                                 */
+/*                                 compile.c                                 */
 /*                                                                           */
-/*                      Top level compiler subroutine                       */
+/*                       Top level compiler subroutine                       */
 /*                                                                           */
 /*                                                                           */
 /*                                                                           */
-/* (C) 2000-2004 Ullrich von Bassewitz                                       */
-/*               Römerstrasse 52                                             */
-/*               D-70794 Filderstadt                                         */
-/* EMail:        uz@cc65.org                                                 */
+/* (C) 2000-2013, Ullrich von Bassewitz                                      */
+/*                Roemerstrasse 52                                           */
+/*                D-70794 Filderstadt                                        */
+/* EMail:         uz@cc65.org                                                */
 /*                                                                           */
 /*                                                                           */
 /* This software is provided 'as-is', without any expressed or implied       */
@@ -46,6 +46,7 @@
 #include "asmlabel.h"
 #include "asmstmt.h"
 #include "codegen.h"
+#include "codeopt.h"
 #include "compile.h"
 #include "declare.h"
 #include "error.h"
@@ -55,6 +56,7 @@
 #include "input.h"
 #include "litpool.h"
 #include "macrotab.h"
+#include "output.h"
 #include "pragma.h"
 #include "preproc.h"
 #include "standard.h"
@@ -63,7 +65,7 @@
 
 
 /*****************************************************************************/
-/*                                  Code                                    */
+/*                                   Code                                    */
 /*****************************************************************************/
 
 
@@ -81,33 +83,34 @@ static void Parse (void)
     /* Parse until end of input */
     while (CurTok.Tok != TOK_CEOF) {
 
-       DeclSpec        Spec;
-       Declaration     Decl;
-       int             NeedStorage;
-
-       /* Check for empty statements */
-       if (CurTok.Tok == TOK_SEMI) {
-           NextToken ();
-           continue;
-       }
-
-       /* Check for an ASM statement (which is allowed also on global level) */
-       if (CurTok.Tok == TOK_ASM) {
-           AsmStatement ();
-           ConsumeSemi ();
-           continue;
-       }
-
-       /* Check for a #pragma */
-       if (CurTok.Tok == TOK_PRAGMA) {
-           DoPragma ();
-           continue;
-       }
-
-               /* Read variable defs and functions */
-       ParseDeclSpec (&Spec, SC_EXTERN | SC_STATIC, T_INT);
-
-       /* Don't accept illegal storage classes */
+        DeclSpec        Spec;
+
+        /* Check for empty statements */
+        if (CurTok.Tok == TOK_SEMI) {
+            NextToken ();
+            continue;
+        }
+
+        /* Disallow ASM statements on global level */
+        if (CurTok.Tok == TOK_ASM) {
+            Error ("__asm__ is not allowed here");
+            /* Parse and remove the statement for error recovery */
+            AsmStatement ();
+            ConsumeSemi ();
+            RemoveGlobalCode ();
+            continue;
+        }
+
+        /* Check for a #pragma */
+        if (CurTok.Tok == TOK_PRAGMA) {
+            DoPragma ();
+            continue;
+        }
+
+        /* Read variable defs and functions */
+        ParseDeclSpec (&Spec, SC_EXTERN | SC_STATIC, T_INT);
+
+        /* Don't accept illegal storage classes */
         if ((Spec.StorageClass & SC_TYPE) == 0) {
             if ((Spec.StorageClass & SC_AUTO) != 0 ||
                 (Spec.StorageClass & SC_REGISTER) != 0) {
@@ -116,106 +119,120 @@ static void Parse (void)
             }
         }
 
-       /* Check if this is only a type declaration */
-       if (CurTok.Tok == TOK_SEMI) {
-           CheckEmptyDecl (&Spec);
-           NextToken ();
-           continue;
-       }
-
-               /* Check if we must reserve storage for the variable. We do
-        * this if we don't had a storage class given ("int i") or
-        * if the storage class is explicitly specified as static.
-        * This means that "extern int i" will not get storage
-        * allocated.
-        */
-       NeedStorage = (Spec.StorageClass & SC_TYPEDEF) == 0 &&
-                     ((Spec.Flags & DS_DEF_STORAGE) != 0  ||
-                     (Spec.StorageClass & (SC_STATIC | SC_EXTERN)) == SC_STATIC);
-
-       /* Read declarations for this type */
-       Entry = 0;
-       comma = 0;
-               while (1) {
-
-           unsigned SymFlags;
-
-           /* Read the next declaration */
-           ParseDecl (&Spec, &Decl, DM_NEED_IDENT);
-           if (Decl.Ident[0] == '\0') {
-               NextToken ();
-               break;
-           }
-
-           /* Get the symbol flags */
-           SymFlags = Spec.StorageClass;
-           if (IsTypeFunc (Decl.Type)) {
-               SymFlags |= SC_FUNC;
-           } else if ((SymFlags & SC_TYPEDEF) == 0) {
-                if ((Spec.Flags & DS_DEF_TYPE) != 0 && IS_Get (&Standard) >= STD_C99) {
-                    Warning ("Implicit `int' is an obsolete feature");
+        /* Check if this is only a type declaration */
+        if (CurTok.Tok == TOK_SEMI) {
+            CheckEmptyDecl (&Spec);
+            NextToken ();
+            continue;
+        }
+
+        /* Read declarations for this type */
+        Entry = 0;
+        comma = 0;
+        while (1) {
+
+            Declaration         Decl;
+
+            /* Read the next declaration */
+            ParseDecl (&Spec, &Decl, DM_NEED_IDENT);
+            if (Decl.Ident[0] == '\0') {
+                NextToken ();
+                break;
+            }
+
+            /* Check if we must reserve storage for the variable. We do this,
+            **
+            **   - if it is not a typedef or function,
+            **   - if we don't had a storage class given ("int i")
+            **     or the storage class is explicitly specified as static.
+            **
+            ** This means that "extern int i;" will not get storage allocated.
+            */
+            if ((Decl.StorageClass & SC_FUNC) != SC_FUNC          &&
+                (Decl.StorageClass & SC_TYPEMASK) != SC_TYPEDEF    &&
+                ((Spec.Flags & DS_DEF_STORAGE) != 0         ||
+                 (Decl.StorageClass & (SC_EXTERN|SC_STATIC)) == SC_STATIC)) {
+
+                /* We will allocate storage */
+                Decl.StorageClass |= SC_STORAGE | SC_DEF;
+            }
+
+            /* If this is a function declarator that is not followed by a comma
+            ** or semicolon, it must be followed by a function body. If this is
+            ** the case, convert an empty parameter list into one accepting no
+            ** parameters (same as void) as required by the standard.
+            */
+            if ((Decl.StorageClass & SC_FUNC) != 0 &&
+                (CurTok.Tok != TOK_COMMA)          &&
+                (CurTok.Tok != TOK_SEMI)) {
+
+                FuncDesc* D = GetFuncDesc (Decl.Type);
+                if (D->Flags & FD_EMPTY) {
+                    D->Flags = (D->Flags & ~(FD_EMPTY | FD_VARIADIC)) | FD_VOID_PARAM;
                 }
-               if (NeedStorage) {
-                   /* We will allocate storage, variable is defined */
-                   SymFlags |= SC_STORAGE | SC_DEF;
-               }
-           }
-
-           /* Add an entry to the symbol table */
-           Entry = AddGlobalSym (Decl.Ident, Decl.Type, SymFlags);
-
-           /* Reserve storage for the variable if we need to */
-                   if (SymFlags & SC_STORAGE) {
-
-               /* Get the size of the variable */
-               unsigned Size = SizeOf (Decl.Type);
-
-               /* Allow initialization */
-               if (CurTok.Tok == TOK_ASSIGN) {
-
-                   /* We cannot initialize types of unknown size, or
-                    * void types in non ANSI mode.
-                    */
-                           if (Size == 0) {
-                       if (!IsTypeVoid (Decl.Type)) {
-                           if (!IsTypeArray (Decl.Type)) {
-                               /* Size is unknown and not an array */
-                               Error ("Variable `%s' has unknown size", Decl.Ident);
-                           }
-                       } else if (IS_Get (&Standard) != STD_CC65) {
-                           /* We cannot declare variables of type void */
-                           Error ("Illegal type for variable `%s'", Decl.Ident);
-                       }
-                   }
-
-                   /* Switch to the data or rodata segment */
-                   if (IsQualConst (Decl.Type)) {
-                       g_userodata ();
-                   } else {
-                       g_usedata ();
-                   }
-
-                   /* Define a label */
-                   g_defgloblabel (Entry->Name);
-
-                   /* Skip the '=' */
-                   NextToken ();
-
-                   /* Parse the initialization */
-                   ParseInit (Entry->Type);
-               } else {
-
-                   if (IsTypeVoid (Decl.Type)) {
-                       /* We cannot declare variables of type void */
-                       Error ("Illegal type for variable `%s'", Decl.Ident);
+            }
+
+            /* Add an entry to the symbol table */
+            Entry = AddGlobalSym (Decl.Ident, Decl.Type, Decl.StorageClass);
+
+            /* Add declaration attributes */
+            SymUseAttr (Entry, &Decl);
+
+            /* Reserve storage for the variable if we need to */
+            if (Decl.StorageClass & SC_STORAGE) {
+
+                /* Get the size of the variable */
+                unsigned Size = SizeOf (Decl.Type);
+
+                /* Allow initialization */
+                if (CurTok.Tok == TOK_ASSIGN) {
+
+                    /* We cannot initialize types of unknown size, or
+                    ** void types in ISO modes.
+                    */
+                    if (Size == 0) {
+                        if (!IsTypeVoid (Decl.Type)) {
+                            if (!IsTypeArray (Decl.Type)) {
+                                /* Size is unknown and not an array */
+                                Error ("Variable `%s' has unknown size", Decl.Ident);
+                            }
+                        } else if (IS_Get (&Standard) != STD_CC65) {
+                            /* We cannot declare variables of type void */
+                            Error ("Illegal type for variable `%s'", Decl.Ident);
+                        }
+                    }
+
+                    /* Switch to the data or rodata segment. For arrays, check
+                     ** the element qualifiers, since not the array but its
+                     ** elements are const.
+                     */
+                    if (IsQualConst (GetBaseElementType (Decl.Type))) {
+                        g_userodata ();
+                    } else {
+                        g_usedata ();
+                    }
+
+                    /* Define a label */
+                    g_defgloblabel (Entry->Name);
+
+                    /* Skip the '=' */
+                    NextToken ();
+
+                    /* Parse the initialization */
+                    ParseInit (Entry->Type);
+                } else {
+
+                    if (IsTypeVoid (Decl.Type)) {
+                        /* We cannot declare variables of type void */
+                        Error ("Illegal type for variable `%s'", Decl.Ident);
                         Entry->Flags &= ~(SC_STORAGE | SC_DEF);
                     } else if (Size == 0) {
-                       /* Size is unknown. Is it an array? */
+                        /* Size is unknown. Is it an array? */
                         if (!IsTypeArray (Decl.Type)) {
                             Error ("Variable `%s' has unknown size", Decl.Ident);
                         }
                         Entry->Flags &= ~(SC_STORAGE | SC_DEF);
-                   }
+                    }
 
                     /* Allocate storage if it is still needed */
                     if (Entry->Flags & SC_STORAGE) {
@@ -229,43 +246,46 @@ static void Parse (void)
                         /* Allocate space for uninitialized variable */
                         g_res (Size);
                     }
-               }
-
-           }
-
-           /* Check for end of declaration list */
-           if (CurTok.Tok == TOK_COMMA) {
-               NextToken ();
-               comma = 1;
-           } else {
-               break;
-           }
-       }
-
-       /* Function declaration? */
-       if (Entry && IsTypeFunc (Entry->Type)) {
-
-           /* Function */
-           if (!comma) {
-               if (CurTok.Tok == TOK_SEMI) {
-                   /* Prototype only */
-                   NextToken ();
-               } else if (Entry) {
-                    /* Function body definition */
+                }
+
+            }
+
+            /* Check for end of declaration list */
+            if (CurTok.Tok == TOK_COMMA) {
+                NextToken ();
+                comma = 1;
+            } else {
+                break;
+            }
+        }
+
+        /* Function declaration? */
+        if (Entry && IsTypeFunc (Entry->Type)) {
+
+            /* Function */
+            if (!comma) {
+                if (CurTok.Tok == TOK_SEMI) {
+                    /* Prototype only */
+                    NextToken ();
+                } else {
+
+                    /* Function body. Check for duplicate function definitions */
                     if (SymIsDef (Entry)) {
                         Error ("Body for function `%s' has already been defined",
                                Entry->Name);
                     }
+
+                    /* Parse the function body */
                     NewFunc (Entry);
-               }
-           }
+                }
+            }
 
-       } else {
+        } else {
 
-           /* Must be followed by a semicolon */
-           ConsumeSemi ();
+            /* Must be followed by a semicolon */
+            ConsumeSemi ();
 
-       }
+        }
     }
 }
 
@@ -280,15 +300,15 @@ void Compile (const char* FileName)
     struct tm*  TM;
 
     /* Since strftime is locale dependent, we need the abbreviated month names
-     * in english.
-     */
+    ** in english.
+    */
     static const char MonthNames[12][4] = {
         "Jan", "Feb", "Mar", "Apr", "May", "Jun",
         "Jul", "Aug", "Sep", "Oct", "Nov", "Dec"
     };
 
     /* Add macros that are always defined */
-    DefineNumericMacro ("__CC65__", VERSION);
+    DefineNumericMacro ("__CC65__", GetVersionAsNumber ());
 
     /* Language standard that is supported */
     DefineNumericMacro ("__CC65_STD_C89__", STD_C89);
@@ -297,21 +317,21 @@ void Compile (const char* FileName)
     DefineNumericMacro ("__CC65_STD__", IS_Get (&Standard));
 
     /* Optimization macros. Since no source code has been parsed for now, the
-     * IS_Get functions access the values in effect now, regardless of any
-     * changes using #pragma later.
-     */
+    ** IS_Get functions access the values in effect now, regardless of any
+    ** changes using #pragma later.
+    */
     if (IS_Get (&Optimize)) {
         long CodeSize = IS_Get (&CodeSizeFactor);
-       DefineNumericMacro ("__OPT__", 1);
-               if (CodeSize > 100) {
-           DefineNumericMacro ("__OPT_i__", CodeSize);
-       }
-       if (IS_Get (&EnableRegVars)) {
-           DefineNumericMacro ("__OPT_r__", 1);
-       }
-               if (IS_Get (&InlineStdFuncs)) {
-           DefineNumericMacro ("__OPT_s__", 1);
-       }
+        DefineNumericMacro ("__OPT__", 1);
+        if (CodeSize > 100) {
+            DefineNumericMacro ("__OPT_i__", CodeSize);
+        }
+        if (IS_Get (&EnableRegVars)) {
+            DefineNumericMacro ("__OPT_r__", 1);
+        }
+        if (IS_Get (&InlineStdFuncs)) {
+            DefineNumericMacro ("__OPT_s__", 1);
+        }
     }
 
     /* __TIME__ and __DATE__ macros */
@@ -323,12 +343,19 @@ void Compile (const char* FileName)
     DefineTextMacro ("__DATE__", DateStr);
     DefineTextMacro ("__TIME__", TimeStr);
 
-    /* Initialize the literal pool */
-    InitLiteralPool ();
+    /* Other standard macros */
+    /* DefineNumericMacro ("__STDC__", 1);      <- not now */
+    DefineNumericMacro ("__STDC_HOSTED__", 1);
 
     /* Create the base lexical level */
     EnterGlobalLevel ();
 
+    /* Create the global code and data segments */
+    CreateGlobalSegments ();
+
+    /* Initialize the literal pool */
+    InitLiteralPool ();
+
     /* Generate the code generator preamble */
     g_preamble ();
 
@@ -338,35 +365,28 @@ void Compile (const char* FileName)
     /* Are we supposed to compile or just preprocess the input? */
     if (PreprocessOnly) {
 
+        /* Open the file */
+        OpenOutputFile ();
+
+        /* Preprocess each line and write it to the output file */
         while (NextLine ()) {
             Preprocess ();
-            printf ("%.*s\n", SB_GetLen (Line), SB_GetConstBuf (Line));
+            WriteOutput ("%.*s\n", (int) SB_GetLen (Line), SB_GetConstBuf (Line));
         }
 
-        if (Debug) {
-            PrintMacroStats (stdout);
-        }
+        /* Close the output file */
+        CloseOutputFile ();
 
     } else {
 
         /* Ok, start the ball rolling... */
         Parse ();
 
-        /* Dump the literal pool. */
-        DumpLiteralPool ();
-
-        /* Write imported/exported symbols */
-        EmitExternals ();
-
-        if (Debug) {
-            PrintLiteralPoolStats (stdout);
-            PrintMacroStats (stdout);
-        }
-
     }
 
-    /* Leave the main lexical level */
-    LeaveGlobalLevel ();
+    if (Debug) {
+        PrintMacroStats (stdout);
+    }
 
     /* Print an error report */
     ErrorReport ();
@@ -374,4 +394,34 @@ void Compile (const char* FileName)
 
 
 
+void FinishCompile (void)
+/* Emit literals, externals, debug info, do cleanup and optimizations */
+{
+    SymTable* SymTab;
+    SymEntry* Func;
+
+    /* Walk over all functions, doing cleanup, optimizations ... */
+    SymTab = GetGlobalSymTab ();
+    Func   = SymTab->SymHead;
+    while (Func) {
+        if (SymIsOutputFunc (Func)) {
+            /* Function which is defined and referenced or extern */
+            MoveLiteralPool (Func->V.F.LitPool);
+            CS_MergeLabels (Func->V.F.Seg->Code);
+            RunOpt (Func->V.F.Seg->Code);
+        }
+        Func = Func->NextSym;
+    }
+
+    /* Output the literal pool */
+    OutputLiteralPool ();
+
+    /* Emit debug infos if enabled */
+    EmitDebugInfo ();
 
+    /* Write imported/exported symbols */
+    EmitExternals ();
+
+    /* Leave the main lexical level */
+    LeaveGlobalLevel ();
+}