]> git.sur5r.net Git - cc65/blobdiff - src/cc65/declare.c
Small but significant shift optimization
[cc65] / src / cc65 / declare.c
index dc8ae24606f6536a15029f6a781bbe4dec286516..e2770e1ca0e9f39429cc46082a5403d1f3618923 100644 (file)
@@ -1,8 +1,35 @@
-/*
- * declare.c
- *
- * Ullrich von Bassewitz, 20.06.1998
- */
+/*****************************************************************************/
+/*                                                                           */
+/*                                declare.c                                 */
+/*                                                                           */
+/*                Parse variable and function declarations                  */
+/*                                                                           */
+/*                                                                           */
+/*                                                                           */
+/* (C) 1998-2002 Ullrich von Bassewitz                                       */
+/*               Wacholderweg 14                                             */
+/*               D-70597 Stuttgart                                           */
+/* EMail:        uz@musoftware.de                                            */
+/*                                                                           */
+/*                                                                           */
+/* This software is provided 'as-is', without any expressed or implied       */
+/* warranty.  In no event will the authors be held liable for any damages    */
+/* arising from the use of this software.                                    */
+/*                                                                           */
+/* Permission is granted to anyone to use this software for any purpose,     */
+/* including commercial applications, and to alter it and redistribute it    */
+/* freely, subject to the following restrictions:                            */
+/*                                                                           */
+/* 1. The origin of this software must not be misrepresented; you must not   */
+/*    claim that you wrote the original software. If you use this software   */
+/*    in a product, an acknowledgment in the product documentation would be  */
+/*    appreciated but is not required.                                       */
+/* 2. Altered source versions must be plainly marked as such, and must not   */
+/*    be misrepresented as being the original software.                      */
+/* 3. This notice may not be removed or altered from any source              */
+/*    distribution.                                                          */
+/*                                                                           */
+/*****************************************************************************/
 
 
 
@@ -51,9 +78,9 @@ static void ParseTypeSpec (DeclSpec* D, int Default);
 static type OptionalQualifiers (type Q)
 /* Read type qualifiers if we have any */
 {
-    while (curtok == TOK_CONST || curtok == TOK_VOLATILE) {
+    while (CurTok.Tok == TOK_CONST || CurTok.Tok == TOK_VOLATILE) {
 
-       switch (curtok) {
+       switch (CurTok.Tok) {
 
            case TOK_CONST:
                if (Q & T_QUAL_CONST) {
@@ -88,7 +115,7 @@ static type OptionalQualifiers (type Q)
 static void optionalint (void)
 /* Eat an optional "int" token */
 {
-    if (curtok == TOK_INT) {
+    if (CurTok.Tok == TOK_INT) {
        /* Skip it */
        NextToken ();
     }
@@ -99,7 +126,7 @@ static void optionalint (void)
 static void optionalsigned (void)
 /* Eat an optional "signed" token */
 {
-    if (curtok == TOK_SIGNED) {
+    if (CurTok.Tok == TOK_SIGNED) {
        /* Skip it */
        NextToken ();
     }
@@ -134,7 +161,7 @@ static void ParseStorageClass (DeclSpec* D, unsigned DefStorage)
     D->Flags &= ~DS_DEF_STORAGE;
 
     /* Check the storage class given */
-    switch (curtok) {
+    switch (CurTok.Tok) {
 
        case TOK_EXTERN:
            D->StorageClass = SC_EXTERN | SC_STATIC;
@@ -178,7 +205,7 @@ static void ParseEnumDecl (void)
     ident Ident;
 
     /* Accept forward definitions */
-    if (curtok != TOK_LCURLY) {
+    if (CurTok.Tok != TOK_LCURLY) {
        return;
     }
 
@@ -187,10 +214,10 @@ static void ParseEnumDecl (void)
 
     /* Read the enum tags */
     EnumVal = 0;
-    while (curtok != TOK_RCURLY) {
+    while (CurTok.Tok != TOK_RCURLY) {
 
        /* We expect an identifier */
-       if (curtok != TOK_IDENT) {
+       if (CurTok.Tok != TOK_IDENT) {
            Error ("Identifier expected");
            continue;
        }
@@ -200,18 +227,18 @@ static void ParseEnumDecl (void)
        NextToken ();
 
        /* Check for an assigned value */
-       if (curtok == TOK_ASSIGN) {
-           struct expent lval;
+       if (CurTok.Tok == TOK_ASSIGN) {
+           ExprDesc lval;
            NextToken ();
-           constexpr (&lval);
-           EnumVal = lval.e_const;
+           ConstExpr (&lval);
+           EnumVal = lval.ConstVal;
        }
 
        /* Add an entry to the symbol table */
-       AddEnumSym (Ident, EnumVal++);
+       AddConstSym (Ident, type_int, SC_ENUM, EnumVal++);
 
        /* Check for end of definition */
-       if (curtok != TOK_COMMA)
+       if (CurTok.Tok != TOK_COMMA)
            break;
        NextToken ();
     }
@@ -230,7 +257,7 @@ static SymEntry* ParseStructDecl (const char* Name, type StructType)
     SymEntry* Entry;
 
 
-    if (curtok != TOK_LCURLY) {
+    if (CurTok.Tok != TOK_LCURLY) {
        /* Just a forward declaration. Try to find a struct with the given
         * name. If there is none, insert a forward declaration into the
         * current lexical level.
@@ -256,7 +283,7 @@ static SymEntry* ParseStructDecl (const char* Name, type StructType)
 
     /* Parse struct fields */
     Size = 0;
-    while (curtok != TOK_RCURLY) {
+    while (CurTok.Tok != TOK_RCURLY) {
 
        /* Get the type of the entry */
        DeclSpec Spec;
@@ -274,7 +301,7 @@ static SymEntry* ParseStructDecl (const char* Name, type StructType)
            AddLocalSym (Decl.Ident, Decl.Type, SC_SFLD, (StructType == T_STRUCT)? Size : 0);
 
            /* Calculate offset of next field/size of the union */
-           Offs = SizeOf (Decl.Type);
+           Offs = CheckedSizeOf (Decl.Type);
            if (StructType == T_STRUCT) {
                Size += Offs;
            } else {
@@ -283,7 +310,7 @@ static SymEntry* ParseStructDecl (const char* Name, type StructType)
                }
            }
 
-           if (curtok != TOK_COMMA)
+           if (CurTok.Tok != TOK_COMMA)
                break;
            NextToken ();
        }
@@ -318,7 +345,7 @@ static void ParseTypeSpec (DeclSpec* D, int Default)
     Qualifiers = OptionalQualifiers (T_QUAL_NONE);
 
     /* Look at the data type */
-    switch (curtok) {
+    switch (CurTok.Tok) {
 
        case TOK_VOID:
            NextToken ();
@@ -334,7 +361,7 @@ static void ParseTypeSpec (DeclSpec* D, int Default)
 
        case TOK_LONG:
            NextToken ();
-           if (curtok == TOK_UNSIGNED) {
+           if (CurTok.Tok == TOK_UNSIGNED) {
                NextToken ();
                optionalint ();
                D->Type[0] = T_ULONG;
@@ -349,7 +376,7 @@ static void ParseTypeSpec (DeclSpec* D, int Default)
 
        case TOK_SHORT:
            NextToken ();
-           if (curtok == TOK_UNSIGNED) {
+           if (CurTok.Tok == TOK_UNSIGNED) {
                NextToken ();
                optionalint ();
                D->Type[0] = T_USHORT;
@@ -370,7 +397,7 @@ static void ParseTypeSpec (DeclSpec* D, int Default)
 
        case TOK_SIGNED:
            NextToken ();
-           switch (curtok) {
+           switch (CurTok.Tok) {
 
                        case TOK_CHAR:
                    NextToken ();
@@ -390,7 +417,7 @@ static void ParseTypeSpec (DeclSpec* D, int Default)
                    optionalint ();
                    D->Type[0] = T_LONG;
                    D->Type[1] = T_END;
-                   break;
+                   break;
 
                case TOK_INT:
                    NextToken ();
@@ -405,7 +432,7 @@ static void ParseTypeSpec (DeclSpec* D, int Default)
 
        case TOK_UNSIGNED:
            NextToken ();
-           switch (curtok) {
+           switch (CurTok.Tok) {
 
                        case TOK_CHAR:
                    NextToken ();
@@ -427,23 +454,23 @@ static void ParseTypeSpec (DeclSpec* D, int Default)
                    D->Type[1] = T_END;
                    break;
 
-               case TOK_INT:
+               case TOK_INT:
                    NextToken ();
                    /* FALL THROUGH */
 
                default:
                    D->Type[0] = T_UINT;
-                   D->Type[1] = T_END;
+                   D->Type[1] = T_END;
                    break;
            }
            break;
 
        case TOK_STRUCT:
        case TOK_UNION:
-           StructType = (curtok == TOK_STRUCT)? T_STRUCT : T_UNION;
+           StructType = (CurTok.Tok == TOK_STRUCT)? T_STRUCT : T_UNION;
            NextToken ();
            /* */
-           if (curtok == TOK_IDENT) {
+           if (CurTok.Tok == TOK_IDENT) {
                strcpy (Ident, CurTok.Ident);
                NextToken ();
            } else {
@@ -461,16 +488,16 @@ static void ParseTypeSpec (DeclSpec* D, int Default)
 
        case TOK_ENUM:
            NextToken ();
-           if (curtok != TOK_LCURLY) {
+           if (CurTok.Tok != TOK_LCURLY) {
                /* Named enum */
-               if (curtok == TOK_IDENT) {
+               if (CurTok.Tok == TOK_IDENT) {
                    /* Find an entry with this name */
                    Entry = FindTagSym (CurTok.Ident);
                    if (Entry) {
                        if (SymIsLocal (Entry) && (Entry->Flags & SC_ENUM) == 0) {
                            Error ("Symbol `%s' is already different kind", Entry->Name);
                        }
-                   } else {
+                   } else {
                        /* Insert entry into table ### */
                    }
                    /* Skip the identifier */
@@ -534,10 +561,10 @@ static void ParseOldStyleParamList (FuncDesc* F)
 /* Parse an old style (K&R) parameter list */
 {
     /* Parse params */
-    while (curtok != TOK_RPAREN) {
+    while (CurTok.Tok != TOK_RPAREN) {
 
        /* List of identifiers expected */
-       if (curtok != TOK_IDENT) {
+       if (CurTok.Tok != TOK_IDENT) {
            Error ("Identifier expected");
        }
 
@@ -551,7 +578,7 @@ static void ParseOldStyleParamList (FuncDesc* F)
        NextToken ();
 
        /* Check for more parameters */
-       if (curtok == TOK_COMMA) {
+       if (CurTok.Tok == TOK_COMMA) {
            NextToken ();
        } else {
            break;
@@ -564,7 +591,7 @@ static void ParseOldStyleParamList (FuncDesc* F)
     ConsumeRParen ();
 
     /* An optional list of type specifications follows */
-    while (curtok != TOK_LCURLY) {
+    while (CurTok.Tok != TOK_LCURLY) {
 
        DeclSpec        Spec;
 
@@ -598,8 +625,8 @@ static void ParseOldStyleParamList (FuncDesc* F)
                }
            }
 
-           if (curtok == TOK_COMMA) {
-               NextToken ();
+           if (CurTok.Tok == TOK_COMMA) {
+               NextToken ();
            } else {
                break;
            }
@@ -617,16 +644,16 @@ static void ParseAnsiParamList (FuncDesc* F)
 /* Parse a new style (ANSI) parameter list */
 {
     /* Parse params */
-    while (curtok != TOK_RPAREN) {
+    while (CurTok.Tok != TOK_RPAREN) {
 
        DeclSpec        Spec;
        Declaration     Decl;
        DeclAttr        Attr;
 
        /* Allow an ellipsis as last parameter */
-       if (curtok == TOK_ELLIPSIS) {
+       if (CurTok.Tok == TOK_ELLIPSIS) {
            NextToken ();
-           F->Flags |= FD_ELLIPSIS;
+           F->Flags |= FD_VARIADIC;
            break;
        }
 
@@ -668,7 +695,7 @@ static void ParseAnsiParamList (FuncDesc* F)
                ++F->ParamCount;
 
        /* Check for more parameters */
-       if (curtok == TOK_COMMA) {
+       if (CurTok.Tok == TOK_COMMA) {
            NextToken ();
        } else {
            break;
@@ -681,7 +708,7 @@ static void ParseAnsiParamList (FuncDesc* F)
     ConsumeRParen ();
 
     /* Check if this is a function definition */
-    if (curtok == TOK_LCURLY) {
+    if (CurTok.Tok == TOK_LCURLY) {
        /* Print an error if in strict ANSI mode and we have unnamed
         * parameters.
         */
@@ -693,7 +720,7 @@ static void ParseAnsiParamList (FuncDesc* F)
 
 
 
-static FuncDesc* ParseFuncDecl (void)
+static FuncDesc* ParseFuncDecl (const DeclSpec* Spec)
 /* Parse the argument list of a function. */
 {
     unsigned Offs;
@@ -706,16 +733,31 @@ static FuncDesc* ParseFuncDecl (void)
     EnterFunctionLevel ();
 
     /* Check for several special parameter lists */
-    if (curtok == TOK_RPAREN) {
+    if (CurTok.Tok == TOK_RPAREN) {
        /* Parameter list is empty */
-       F->Flags |= (FD_EMPTY | FD_ELLIPSIS);
-    } else if (curtok == TOK_VOID && nxttok == TOK_RPAREN) {
+       F->Flags |= (FD_EMPTY | FD_VARIADIC);
+    } else if (CurTok.Tok == TOK_VOID && NextTok.Tok == TOK_RPAREN) {
        /* Parameter list declared as void */
        NextToken ();
        F->Flags |= FD_VOID_PARAM;
-    } else if (curtok == TOK_IDENT && (nxttok == TOK_COMMA || nxttok == TOK_RPAREN)) {
-       /* Old style (K&R) function. Assume variable param list. */
-               F->Flags |= (FD_OLDSTYLE | FD_ELLIPSIS);
+    } else if (CurTok.Tok == TOK_IDENT &&
+              (NextTok.Tok == TOK_COMMA || NextTok.Tok == TOK_RPAREN)) {
+       /* If the identifier is a typedef, we have a new style parameter list,
+        * if it's some other identifier, it's an old style parameter list.
+        */
+       Sym = FindSym (CurTok.Ident);
+       if (Sym == 0 || !IsTypeDef (Sym)) {
+           /* Old style (K&R) function. Assume variable param list. */
+           F->Flags |= (FD_OLDSTYLE | FD_VARIADIC);
+
+           /* Check for an implicit int return in the K&R function */
+           if ((Spec->Flags & DS_DEF_TYPE) != 0 &&
+               Spec->Type[0] == T_INT   &&
+               Spec->Type[1] == T_END) {
+               /* Function has an implicit int return */
+               F->Flags |= FD_OLDSTYLE_INTRET;
+           }
+       }
     }
 
     /* Parse params */
@@ -730,10 +772,10 @@ static FuncDesc* ParseFuncDecl (void)
     /* Assign offsets. If the function has a variable parameter list,
      * there's one additional byte (the arg size).
      */
-    Offs = (F->Flags & FD_ELLIPSIS)? 1 : 0;
+    Offs = (F->Flags & FD_VARIADIC)? 1 : 0;
     Sym = GetSymTab()->SymTail;
     while (Sym) {
-       unsigned Size = SizeOf (Sym->Type);
+       unsigned Size = CheckedSizeOf (Sym->Type);
        Sym->V.Offs = Offs;
                Offs += Size;
        F->ParamSize += Size;
@@ -749,34 +791,36 @@ static FuncDesc* ParseFuncDecl (void)
 
 
 
-static void Decl (Declaration* D, unsigned Mode)
+static void Decl (const DeclSpec* Spec, Declaration* D, unsigned Mode)
 /* Recursively process declarators. Build a type array in reverse order. */
 {
 
-    if (curtok == TOK_STAR) {
-       type T = T_PTR;
+    if (CurTok.Tok == TOK_STAR) {
+       type T = T_PTR;
                NextToken ();
-       /* Allow optional const or volatile qualifiers */
-       T |= OptionalQualifiers (T_QUAL_NONE);
-               Decl (D, Mode);
+       /* Allow optional const or volatile qualifiers */
+       T |= OptionalQualifiers (T_QUAL_NONE);
+               Decl (Spec, D, Mode);
                *D->T++ = T;
                return;
-    } else if (curtok == TOK_LPAREN) {
+    } else if (CurTok.Tok == TOK_LPAREN) {
                NextToken ();
-               Decl (D, Mode);
+               Decl (Spec, D, Mode);
                ConsumeRParen ();
-    } else if (curtok == TOK_FASTCALL) {
+    } else if (CurTok.Tok == TOK_FASTCALL) {
        /* Remember the current type pointer */
        type* T = D->T;
        /* Skip the fastcall token */
        NextToken ();
        /* Parse the function */
-       Decl (D, Mode);
+       Decl (Spec, D, Mode);
        /* Set the fastcall flag */
-       if (!IsTypeFunc (T)) {
+       if (!IsTypeFunc (T) && !IsTypeFuncPtr (T)) {
            Error ("__fastcall__ modifier applied to non function");
+       } else if (IsVariadicFunc (T)) {
+           Error ("Cannot apply __fastcall__ to functions with variable parameter list");
        } else {
-           FuncDesc* F = DecodePtr (T+1);
+           FuncDesc* F = GetFuncDesc (T);
                    F->Flags |= FD_FASTCALL;
        }
        return;
@@ -785,15 +829,15 @@ static void Decl (Declaration* D, unsigned Mode)
                 *  - Mode == DM_NEED_IDENT means:
         *      we *must* have a type and a variable identifer.
         *  - Mode == DM_NO_IDENT means:
-        *      we must have a type but no variable identifer
+        *      we must have a type but no variable identifer
         *      (if there is one, it's not read).
         *  - Mode == DM_ACCEPT_IDENT means:
-        *      we *may* have an identifier. If there is an identifier,
-        *      it is read, but it is no error, if there is none.
+        *      we *may* have an identifier. If there is an identifier,
+        *      it is read, but it is no error, if there is none.
         */
        if (Mode == DM_NO_IDENT) {
            D->Ident[0] = '\0';
-       } else if (curtok == TOK_IDENT) {
+       } else if (CurTok.Tok == TOK_IDENT) {
                    strcpy (D->Ident, CurTok.Ident);
            NextToken ();
        } else {
@@ -801,17 +845,16 @@ static void Decl (Declaration* D, unsigned Mode)
                Error ("Identifier expected");
            }
            D->Ident[0] = '\0';
-           return;
        }
     }
 
-    while (curtok == TOK_LBRACK || curtok == TOK_LPAREN) {
-               if (curtok == TOK_LPAREN) {
+    while (CurTok.Tok == TOK_LBRACK || CurTok.Tok == TOK_LPAREN) {
+               if (CurTok.Tok == TOK_LPAREN) {
                    /* Function declaration */
            FuncDesc* F;
                    NextToken ();
            /* Parse the function declaration */
-                   F = ParseFuncDecl ();
+                   F = ParseFuncDecl (Spec);
            *D->T++ = T_FUNC;
            EncodePtr (D->T, F);
            D->T += DECODE_SIZE;
@@ -820,10 +863,10 @@ static void Decl (Declaration* D, unsigned Mode)
                    unsigned long Size = 0;
                    NextToken ();
            /* Read the size if it is given */
-                   if (curtok != TOK_RBRACK) {
-               struct expent lval;
-                       constexpr (&lval);
-                       Size = lval.e_const;
+                   if (CurTok.Tok != TOK_RBRACK) {
+               ExprDesc lval;
+                       ConstExpr (&lval);
+                       Size = lval.ConstVal;
                    }
                    ConsumeRBrack ();
                    *D->T++ = T_ARRAY;
@@ -871,18 +914,18 @@ void ParseDecl (const DeclSpec* Spec, Declaration* D, unsigned Mode)
     InitDeclaration (D);
 
     /* Get additional declarators and the identifier */
-    Decl (D, Mode);
+    Decl (Spec, D, Mode);
 
     /* Add the base type. */
     TypeCpy (D->T, Spec->Type);
 
     /* Check the size of the generated type */
     if (!IsTypeFunc (D->Type) && !IsTypeVoid (D->Type) && SizeOf (D->Type) >= 0x10000) {
-       if (D->Ident[0] != '\0') {
-           Error ("Size of `%s' is invalid", D->Ident);
-       } else {
-           Error ("Invalid size");
-       }
+       if (D->Ident[0] != '\0') {
+           Error ("Size of `%s' is invalid", D->Ident);
+       } else {
+           Error ("Invalid size");
+       }
     }
 }
 
@@ -919,19 +962,19 @@ void CheckEmptyDecl (const DeclSpec* D)
 static void ParseVoidInit (void)
 /* Parse an initialization of a void variable (special cc65 extension) */
 {
-    struct expent lval;
+    ExprDesc lval;
 
     /* Allow an arbitrary list of values */
     ConsumeLCurly ();
     do {
-       constexpr (&lval);
-       switch (lval.e_tptr[0]) {
+       ConstExpr (&lval);
+       switch (lval.Type[0]) {
 
            case T_SCHAR:
            case T_UCHAR:
-               if ((lval.e_flags & E_MCTYPE) == E_TCONST) {
+               if ((lval.Flags & E_MCTYPE) == E_TCONST) {
                    /* Make it byte sized */
-                   lval.e_const &= 0xFF;
+                   lval.ConstVal &= 0xFF;
                }
                DefineData (&lval);
                break;
@@ -942,9 +985,9 @@ static void ParseVoidInit (void)
            case T_UINT:
            case T_PTR:
            case T_ARRAY:
-               if ((lval.e_flags & E_MCTYPE) == E_TCONST) {
+               if ((lval.Flags & E_MCTYPE) == E_TCONST) {
                    /* Make it word sized */
-                   lval.e_const &= 0xFFFF;
+                   lval.ConstVal &= 0xFFFF;
                }
                DefineData (&lval);
                break;
@@ -960,12 +1003,12 @@ static void ParseVoidInit (void)
 
        }
 
-       if (curtok != TOK_COMMA) {
+       if (CurTok.Tok != TOK_COMMA) {
            break;
        }
        NextToken ();
 
-    } while (curtok != TOK_RCURLY);
+    } while (CurTok.Tok != TOK_RCURLY);
 
     ConsumeRCurly ();
 }
@@ -996,14 +1039,14 @@ static void ParseStructInit (type* Type)
 
     /* Get a pointer to the list of symbols */
     Entry = Tab->SymHead;
-    while (curtok != TOK_RCURLY) {
+    while (CurTok.Tok != TOK_RCURLY) {
        if (Entry == 0) {
            Error ("Too many initializers");
            return;
        }
        ParseInit (Entry->Type);
        Entry = Entry->NextSym;
-       if (curtok != TOK_COMMA)
+       if (CurTok.Tok != TOK_COMMA)
            break;
        NextToken ();
     }
@@ -1013,7 +1056,7 @@ static void ParseStructInit (type* Type)
 
     /* If there are struct fields left, reserve additional storage */
     while (Entry) {
-       g_zerobytes (SizeOf (Entry->Type));
+       g_zerobytes (CheckedSizeOf (Entry->Type));
        Entry = Entry->NextSym;
     }
 }
@@ -1023,20 +1066,20 @@ static void ParseStructInit (type* Type)
 void ParseInit (type* T)
 /* Parse initialization of variables. */
 {
-    int count;
-    struct expent lval;
+    ExprDesc lval;
     type* t;
     const char* str;
-    int sz;
+    int Count;
+    int Size;
 
     switch (UnqualifiedType (*T)) {
 
        case T_SCHAR:
        case T_UCHAR:
-           constexpr (&lval);
-           if ((lval.e_flags & E_MCTYPE) == E_TCONST) {
+           ConstExpr (&lval);
+           if ((lval.Flags & E_MCTYPE) == E_TCONST) {
                /* Make it byte sized */
-               lval.e_const &= 0xFF;
+               lval.ConstVal &= 0xFF;
            }
            assignadjust (T, &lval);
            DefineData (&lval);
@@ -1047,10 +1090,10 @@ void ParseInit (type* T)
        case T_INT:
        case T_UINT:
        case T_PTR:
-           constexpr (&lval);
-           if ((lval.e_flags & E_MCTYPE) == E_TCONST) {
+           ConstExpr (&lval);
+           if ((lval.Flags & E_MCTYPE) == E_TCONST) {
                /* Make it word sized */
-               lval.e_const &= 0xFFFF;
+               lval.ConstVal &= 0xFFFF;
            }
            assignadjust (T, &lval);
            DefineData (&lval);
@@ -1058,42 +1101,42 @@ void ParseInit (type* T)
 
        case T_LONG:
        case T_ULONG:
-           constexpr (&lval);
-           if ((lval.e_flags & E_MCTYPE) == E_TCONST) {
+           ConstExpr (&lval);
+           if ((lval.Flags & E_MCTYPE) == E_TCONST) {
                /* Make it long sized */
-               lval.e_const &= 0xFFFFFFFF;
+               lval.ConstVal &= 0xFFFFFFFF;
            }
            assignadjust (T, &lval);
            DefineData (&lval);
            break;
 
        case T_ARRAY:
-           sz = Decode (T + 1);
+           Size = Decode (T + 1);
            t = T + DECODE_SIZE + 1;
-                   if (IsTypeChar(t) && curtok == TOK_SCONST) {
-               str = GetLiteral (curval);
-               count = strlen (str) + 1;
-               TranslateLiteralPool (curval);  /* Translate into target charset */
-               g_defbytes (str, count);
-               ResetLiteralOffs (curval);      /* Remove string from pool */
+                   if (IsTypeChar(t) && CurTok.Tok == TOK_SCONST) {
+               str = GetLiteral (CurTok.IVal);
+               Count = strlen (str) + 1;
+               TranslateLiteralPool (CurTok.IVal);     /* Translate into target charset */
+               g_defbytes (str, Count);
+               ResetLiteralPoolOffs (CurTok.IVal);     /* Remove string from pool */
                NextToken ();
            } else {
                ConsumeLCurly ();
-               count = 0;
-               while (curtok != TOK_RCURLY) {
+               Count = 0;
+               while (CurTok.Tok != TOK_RCURLY) {
                    ParseInit (T + DECODE_SIZE + 1);
-                   ++count;
-                   if (curtok != TOK_COMMA)
+                   ++Count;
+                   if (CurTok.Tok != TOK_COMMA)
                        break;
                    NextToken ();
                }
                ConsumeRCurly ();
            }
-           if (sz == 0) {
-               Encode (T + 1, count);
-           } else if (count < sz) {
-               g_zerobytes ((sz - count) * SizeOf (T + DECODE_SIZE + 1));
-           } else if (count > sz) {
+           if (Size == 0) {
+               Encode (T + 1, Count);
+           } else if (Count < Size) {
+               g_zerobytes ((Size - Count) * CheckedSizeOf (T + DECODE_SIZE + 1));
+           } else if (Count > Size) {
                Error ("Too many initializers");
            }
            break;