Fixed two compiler warnings.

[cc65] / src / cc65 / compile.c

diff --git a/src/cc65/compile.c b/src/cc65/compile.c
index 9d92b4acec504f886cf3546b53c911953c8bbb39..d7167706b21de9ca4ec2e9ba05944fe201ee3c29 100644 (file)
--- a/src/cc65/compile.c
+++ b/src/cc65/compile.c
@@ -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"
@@ -90,10 +91,13 @@ static void Parse (void)
            continue;
        }
 
-       /* Check for an ASM statement (which is allowed also on global level) */
+               /* 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;
        }
 
@@ -137,15 +141,20 @@ static void Parse (void)
            }
 
             /* 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 if the storage class is explicitly
-             * specified as static. This means that "extern int i" will not
-             * get storage allocated.
+             *
+             *   - if it is not a typedef or function,
+             *   - if we don't had a storage class given ("int i")
+             *   - if the storage class is explicitly specified as static,
+             *   - or if there is an initialization.
+             *
+             * This means that "extern int i;" will not get storage allocated.
              */
            if ((Decl.StorageClass & SC_FUNC) != SC_FUNC          &&
                 (Decl.StorageClass & SC_TYPEDEF) != SC_TYPEDEF    &&
-                ((Spec.Flags & DS_DEF_STORAGE) != 0  ||
-                 (Decl.StorageClass & (SC_STATIC | SC_EXTERN)) == SC_STATIC)) {
+                ((Spec.Flags & DS_DEF_STORAGE) != 0         ||
+                 (Decl.StorageClass & (SC_EXTERN|SC_STATIC)) == SC_STATIC ||
+                 ((Decl.StorageClass & SC_EXTERN) != 0 &&
+                  CurTok.Tok == TOK_ASSIGN))) {
 
                 /* We will allocate storage */
                Decl.StorageClass |= SC_STORAGE | SC_DEF;
@@ -373,35 +382,54 @@ void Compile (const char* FileName)
         /* Close the output file */
         CloseOutputFile ();
 
-        if (Debug) {
-            PrintMacroStats (stdout);
-        }
-
     } else {
 
         /* Ok, start the ball rolling... */
         Parse ();
 
-        /* Dump the literal pool. */
-        DumpLiteralPool ();
+    }
+
+    if (Debug) {
+        PrintMacroStats (stdout);
+    }
+
+    /* Print an error report */
+    ErrorReport ();
+}
 
-        /* Write imported/exported symbols */
-        EmitExternals ();
 
-        if (Debug) {
-            PrintLiteralPoolStats (stdout);
-            PrintMacroStats (stdout);
-        }
 
+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 ();
-
-    /* Print an error report */
-    ErrorReport ();
 }
 
 
 
-