]> git.sur5r.net Git - cc65/blobdiff - src/cc65/expr.c
Fixed a bug
[cc65] / src / cc65 / expr.c
index 11c43b9730a2f0095febc67d4a02de3282305243..92d6013d80c678e0bc59d5f5a24fe1940906f87a 100644 (file)
@@ -8,17 +8,18 @@
 
 #include <stdio.h>
 #include <stdlib.h>
-#include <string.h>
 
 /* common */
 #include "check.h"
+#include "debugflag.h"
 #include "xmalloc.h"
 
 /* cc65 */
 #include "asmcode.h"
 #include "asmlabel.h"
+#include "asmstmt.h"
+#include "assignment.h"
 #include "codegen.h"
-#include "datatype.h"
 #include "declare.h"
 #include "error.h"
 #include "funcdesc.h"
@@ -30,6 +31,7 @@
 #include "scanner.h"
 #include "stdfunc.h"
 #include "symtab.h"
+#include "typecast.h"
 #include "typecmp.h"
 #include "expr.h"
 
@@ -46,8 +48,8 @@
 
 /* Map a generator function and its attributes to a token */
 typedef struct {
-    unsigned char Tok;                 /* Token to map to */
-    unsigned char Flags;               /* Flags for generator function */
+    token_t       Tok;                 /* Token to map to */
+    unsigned      Flags;               /* Flags for generator function */
     void                 (*Func) (unsigned, unsigned long);    /* Generator func */
 } GenDesc;
 
@@ -85,8 +87,11 @@ static GenDesc GenOASGN  = { TOK_OR_ASSIGN,  GEN_NOPUSH,     g_or  };
 
 
 
-static int hie10 (struct expent* lval);
-/* Handle ++, --, !, unary - etc. */
+static int hie0 (ExprDesc *lval);
+/* Parse comma operator. */
+
+static int expr (int (*func) (ExprDesc*), ExprDesc *lval);
+/* Expression parser; func is either hie0 or hie1. */
 
 
 
@@ -114,12 +119,12 @@ static unsigned GlobalModeFlags (unsigned flags)
 
 
 
-static int IsNullPtr (struct expent* lval)
+static int IsNullPtr (ExprDesc* lval)
 /* Return true if this is the NULL pointer constant */
 {
-    return (IsClassInt (lval->e_tptr) &&       /* Is it an int? */
-                   lval->e_flags == E_MCONST &&        /* Is it constant? */
-           lval->e_const == 0);                /* And is it's value zero? */
+    return (IsClassInt (lval->Type) &&                 /* Is it an int? */
+                   lval->Flags == E_MCONST &&          /* Is it constant? */
+           lval->ConstVal == 0);               /* And is it's value zero? */
 }
 
 
@@ -149,7 +154,7 @@ static type* promoteint (type* lhst, type* rhst)
 
 
 
-static unsigned typeadjust (struct expent* lhs, struct expent* rhs, int NoPush)
+static unsigned typeadjust (ExprDesc* lhs, ExprDesc* rhs, int NoPush)
 /* Adjust the two values for a binary operation. lhs is expected on stack or
  * to be constant, rhs is expected to be in the primary register or constant.
  * The function will put the type of the result into lhs and return the
@@ -163,12 +168,12 @@ static unsigned typeadjust (struct expent* lhs, struct expent* rhs, int NoPush)
     unsigned flags;
 
     /* Get the type strings */
-    type* lhst = lhs->e_tptr;
-    type* rhst = rhs->e_tptr;
+    type* lhst = lhs->Type;
+    type* rhst = rhs->Type;
 
     /* Generate type adjustment code if needed */
     ltype = TypeOf (lhst);
-    if (lhs->e_flags == E_MCONST) {
+    if (lhs->Flags == E_MCONST) {
        ltype |= CF_CONST;
     }
     if (NoPush) {
@@ -176,13 +181,13 @@ static unsigned typeadjust (struct expent* lhs, struct expent* rhs, int NoPush)
        ltype |= CF_REG;
     }
     rtype = TypeOf (rhst);
-    if (rhs->e_flags == E_MCONST) {
+    if (rhs->Flags == E_MCONST) {
        rtype |= CF_CONST;
     }
     flags = g_typeadjust (ltype, rtype);
 
     /* Set the type of the result */
-    lhs->e_tptr = promoteint (lhst, rhst);
+    lhs->Type = promoteint (lhst, rhst);
 
     /* Return the code generator flags */
     return flags;
@@ -190,7 +195,7 @@ static unsigned typeadjust (struct expent* lhs, struct expent* rhs, int NoPush)
 
 
 
-unsigned assignadjust (type* lhst, struct expent* rhs)
+unsigned assignadjust (type* lhst, ExprDesc* rhs)
 /* Adjust the type of the right hand expression so that it can be assigned to
  * the type on the left hand side. This function is used for assignment and
  * for converting parameters in a function call. It returns the code generator
@@ -201,13 +206,13 @@ unsigned assignadjust (type* lhst, struct expent* rhs)
     /* Get the type of the right hand side. Treat function types as
      * pointer-to-function
      */
-    type* rhst = rhs->e_tptr;
+    type* rhst = rhs->Type;
     if (IsTypeFunc (rhst)) {
        rhst = PointerTo (rhst);
     }
 
     /* After calling this function, rhs will have the type of the lhs */
-    rhs->e_tptr = lhst;
+    rhs->Type = lhst;
 
     /* First, do some type checking */
     if (IsTypeVoid (lhst) || IsTypeVoid (rhst)) {
@@ -222,14 +227,12 @@ unsigned assignadjust (type* lhst, struct expent* rhs)
                } else if (!IsClassInt (rhst)) {
            Error ("Incompatible types");
        } else {
-           /* Adjust the int types. To avoid manipulation of TOS mark lhs
-            * as const.
-            */
+           /* Convert the rhs to the type of the lhs. */
            unsigned flags = TypeOf (rhst);
-                   if (rhs->e_flags & E_MCONST) {
+                   if (rhs->Flags == E_MCONST) {
                flags |= CF_CONST;
            }
-           return g_typeadjust (TypeOf (lhst) | CF_CONST, flags);
+                   return g_typecast (TypeOf (lhst), flags);
         }
     } else if (IsClassPtr (lhst)) {
        if (IsClassPtr (rhst)) {
@@ -239,25 +242,25 @@ unsigned assignadjust (type* lhst, struct expent* rhs)
             *   - the lhs pointer is a void pointer.
             */
            if (!IsTypeVoid (Indirect (lhst)) && !IsTypeVoid (Indirect (rhst))) {
-               /* Compare the types */
-               switch (TypeCmp (lhst, rhst)) {
+               /* Compare the types */
+               switch (TypeCmp (lhst, rhst)) {
 
-                   case TC_INCOMPATIBLE:
-                       Error ("Incompatible pointer types");
-                       break;
+                   case TC_INCOMPATIBLE:
+                       Error ("Incompatible pointer types");
+                       break;
 
-                   case TC_QUAL_DIFF:
-                       Error ("Pointer types differ in type qualifiers");
-                       break;
+                   case TC_QUAL_DIFF:
+                       Error ("Pointer types differ in type qualifiers");
+                       break;
 
-                   default:
-                       /* Ok */
-                       break;
-               }
+                   default:
+                       /* Ok */
+                       break;
+               }
            }
        } else if (IsClassInt (rhst)) {
            /* Int to pointer assignment is valid only for constant zero */
-           if ((rhs->e_flags & E_MCONST) == 0 || rhs->e_const != 0) {
+           if (rhs->Flags != E_MCONST || rhs->ConstVal != 0) {
                Warning ("Converting integer to pointer without a cast");
            }
        } else if (IsTypeFuncPtr (lhst) && IsTypeFunc(rhst)) {
@@ -280,16 +283,16 @@ unsigned assignadjust (type* lhst, struct expent* rhs)
 
 
 
-void DefineData (struct expent* lval)
+void DefineData (ExprDesc* Expr)
 /* Output a data definition for the given expression */
 {
-    unsigned flags = lval->e_flags;
+    unsigned Flags = Expr->Flags;
 
-    switch (flags & E_MCTYPE) {
+    switch (Flags & E_MCTYPE) {
 
                case E_TCONST:
            /* Number */
-           g_defdata (TypeOf (lval->e_tptr) | CF_CONST, lval->e_const, 0);
+           g_defdata (TypeOf (Expr->Type) | CF_CONST, Expr->ConstVal, 0);
                    break;
 
        case E_TREGISTER:
@@ -304,33 +307,33 @@ void DefineData (struct expent* lval)
                case E_TGLAB:
         case E_TLLAB:
                    /* Local or global symbol */
-           g_defdata (GlobalModeFlags (flags), lval->e_name, lval->e_const);
+           g_defdata (GlobalModeFlags (Flags), Expr->Name, Expr->ConstVal);
            break;
 
                case E_TLIT:
            /* a literal of some kind */
-                   g_defdata (CF_STATIC, LiteralLabel, lval->e_const);
+                   g_defdata (CF_STATIC, LiteralPoolLabel, Expr->ConstVal);
                    break;
 
                default:
-           Internal ("Unknown constant type: %04X", flags);
+           Internal ("Unknown constant type: %04X", Flags);
     }
 }
 
 
 
-static void lconst (unsigned flags, struct expent* lval)
-/* Load primary reg with some constant value. */
+static void LoadConstant (unsigned Flags, ExprDesc* Expr)
+/* Load the primary register with some constant value. */
 {
-    switch (lval->e_flags & E_MCTYPE) {
+    switch (Expr->Flags & E_MCTYPE) {
 
        case E_TLOFFS:
-                   g_leasp (lval->e_const);
+                   g_leasp (Expr->ConstVal);
            break;
 
                case E_TCONST:
            /* Number constant */
-                   g_getimmed (flags | TypeOf (lval->e_tptr) | CF_CONST, lval->e_const, 0);
+                   g_getimmed (Flags | TypeOf (Expr->Type) | CF_CONST, Expr->ConstVal, 0);
                    break;
 
        case E_TREGISTER:
@@ -345,24 +348,24 @@ static void lconst (unsigned flags, struct expent* lval)
                case E_TGLAB:
         case E_TLLAB:
                    /* Local or global symbol, load address */
-           flags |= GlobalModeFlags (lval->e_flags);
-           flags &= ~CF_CONST;
-           g_getimmed (flags, lval->e_name, lval->e_const);
+           Flags |= GlobalModeFlags (Expr->Flags);
+           Flags &= ~CF_CONST;
+           g_getimmed (Flags, Expr->Name, Expr->ConstVal);
                    break;
 
                case E_TLIT:
            /* Literal string */
-           g_getimmed (CF_STATIC, LiteralLabel, lval->e_const);
+           g_getimmed (CF_STATIC, LiteralPoolLabel, Expr->ConstVal);
                    break;
 
                default:
-           Internal ("Unknown constant type: %04X", lval->e_flags);
+           Internal ("Unknown constant type: %04X", Expr->Flags);
     }
 }
 
 
 
-static int kcalc (int tok, long val1, long val2)
+static int kcalc (token_t tok, long val1, long val2)
 /* Calculate an operation with left and right operand constant. */
 {
     switch (tok) {
@@ -410,9 +413,10 @@ static int kcalc (int tok, long val1, long val2)
 
 
 
-static GenDesc* FindGen (int Tok, GenDesc** Table)
+static const GenDesc* FindGen (token_t Tok, const GenDesc** Table)
+/* Find a token in a generator table */
 {
-    GenDesc* G;
+    const GenDesc* G;
     while ((G = *Table) != 0) {
        if (G->Tok == Tok) {
            return G;
@@ -431,18 +435,17 @@ static int istypeexpr (void)
 {
     SymEntry* Entry;
 
-    return curtok == TOK_LPAREN && (
-                   (nxttok >= TOK_FIRSTTYPE && nxttok <= TOK_LASTTYPE) ||
-           (nxttok == TOK_CONST)                               ||
-                   (nxttok  == TOK_IDENT                               &&
-           (Entry = FindSym (NextTok.Ident)) != 0              &&
-           IsTypeDef (Entry))
-           );
+    return CurTok.Tok == TOK_LPAREN && (
+                  (NextTok.Tok >= TOK_FIRSTTYPE && NextTok.Tok <= TOK_LASTTYPE) ||
+          (NextTok.Tok == TOK_CONST)                                    ||
+                  (NextTok.Tok  == TOK_IDENT                                    &&
+          (Entry = FindSym (NextTok.Ident)) != 0                        &&
+          SymIsTypeDef (Entry)));
 }
 
 
 
-static void PushAddr (struct expent* lval)
+void PushAddr (ExprDesc* lval)
 /* If the expression contains an address that was somehow evaluated,
  * push this address on the stack. This is a helper function for all
  * sorts of implicit or explicit assignment functions where the lvalue
@@ -450,114 +453,161 @@ static void PushAddr (struct expent* lval)
  */
 {
     /* Get the address on stack if needed */
-    if (lval->e_flags != E_MREG && (lval->e_flags & E_MEXPR)) {
-       /* Push the address (always a pointer) */
-       g_push (CF_PTR, 0);
+    if (lval->Flags != E_MREG && (lval->Flags & E_MEXPR)) {
+       /* Push the address (always a pointer) */
+       g_push (CF_PTR, 0);
+    }
+}
+
+
+
+void ConstSubExpr (int (*F) (ExprDesc*), ExprDesc* Expr)
+/* Will evaluate an expression via the given function. If the result is not
+ * a constant, a diagnostic will be printed, and the value is replaced by
+ * a constant one to make sure there are no internal errors that result
+ * from this input error.
+ */
+{
+    InitExprDesc (Expr);
+    if (F (Expr) != 0 || Expr->Flags != E_MCONST) {
+               Error ("Constant expression expected");
+               /* To avoid any compiler errors, make the expression a valid const */
+       MakeConstIntExpr (Expr, 1);
+    }
+}
+
+
+
+void CheckBoolExpr (ExprDesc* lval)
+/* Check if the given expression is a boolean expression, output a diagnostic
+ * if not.
+ */
+{
+    /* If it's an integer, it's ok. If it's not an integer, but a pointer,
+     * the pointer used in a boolean context is also ok
+     */
+    if (!IsClassInt (lval->Type) && !IsClassPtr (lval->Type)) {
+       Error ("Boolean expression expected");
+       /* To avoid any compiler errors, make the expression a valid int */
+       MakeConstIntExpr (lval, 1);
     }
 }
 
 
 
 /*****************************************************************************/
-/*                                  code                                    */
+/*                                  code                                    */
 /*****************************************************************************/
 
 
 
-void exprhs (unsigned flags, int k, struct expent *lval)
+void exprhs (unsigned flags, int k, ExprDesc* lval)
 /* Put the result of an expression into the primary register */
 {
     int f;
 
-    f = lval->e_flags;
+    f = lval->Flags;
     if (k) {
                /* Dereferenced lvalue */
-       flags |= TypeOf (lval->e_tptr);
-       if (lval->e_test & E_FORCETEST) {
+               flags |= TypeOf (lval->Type);
+       if (lval->Test & E_FORCETEST) {
            flags |= CF_TEST;
-           lval->e_test &= ~E_FORCETEST;
+           lval->Test &= ~E_FORCETEST;
        }
                if (f & E_MGLOBAL) {    /* ref to globalvar */
                    /* Generate code */
            flags |= GlobalModeFlags (f);
-                   g_getstatic (flags, lval->e_name, lval->e_const);
+                   g_getstatic (flags, lval->Name, lval->ConstVal);
                } else if (f & E_MLOCAL) {
            /* ref to localvar */
-                   g_getlocal (flags, lval->e_const);
+                   g_getlocal (flags, lval->ConstVal);
        } else if (f & E_MCONST) {
            /* ref to absolute address */
-           g_getstatic (flags | CF_ABSOLUTE, lval->e_const, 0);
+           g_getstatic (flags | CF_ABSOLUTE, lval->ConstVal, 0);
        } else if (f == E_MEOFFS) {
-           g_getind (flags, lval->e_const);
+           g_getind (flags, lval->ConstVal);
        } else if (f != E_MREG) {
            g_getind (flags, 0);
        }
     } else if (f == E_MEOFFS) {
        /* reference not storable */
-       flags |= TypeOf (lval->e_tptr);
-               g_inc (flags | CF_CONST, lval->e_const);
+       flags |= TypeOf (lval->Type);
+               g_inc (flags | CF_CONST, lval->ConstVal);
     } else if ((f & E_MEXPR) == 0) {
        /* Constant of some sort, load it into the primary */
-       lconst (flags, lval);
+       LoadConstant (flags, lval);
     }
-    if (lval->e_test & E_FORCETEST) {  /* we testing this value? */
-       /* debug... */
-       AddCodeHint ("forcetest");
-       flags |= TypeOf (lval->e_tptr);
-               g_test (flags);                 /* yes, force a test */
-               lval->e_test &= ~E_FORCETEST;
+    /* Are we testing this value? */
+    if (lval->Test & E_FORCETEST) {
+        /* Yes, force a test */
+       flags |= TypeOf (lval->Type);
+               g_test (flags);
+               lval->Test &= ~E_FORCETEST;
     }
 }
 
 
-static void callfunction (struct expent* lval)
-/* Perform a function call.  Called from hie11, this routine will
- * either call the named function, or if the supplied ptr is zero,
- * will call the contents of P.
+
+static unsigned FunctionParamList (FuncDesc* Func)
+/* Parse a function parameter list and pass the parameters to the called
+ * function. Depending on several criteria this may be done by just pushing
+ * each parameter separately, or creating the parameter frame once and then
+ * storing into this frame.
+ * The function returns the size of the parameters pushed.
  */
 {
-    struct expent lval2;
-    FuncDesc*    Func;         /* Function descriptor */
-    int                  Ellipsis;     /* True if we have an open param list */
-    SymEntry*    Param;        /* Current formal parameter */
-    unsigned     ParamCount;   /* Actual parameter count */
-    unsigned     ParamSize;    /* Number of parameter bytes */
-    unsigned     Flags;
-    unsigned     CFlags;
-    CodeMark     Mark;
-
-
-    /* Get a pointer to the function descriptor from the type string */
-    Func = GetFuncDesc (lval->e_tptr);
-
-    /* Initialize vars to keep gcc silent */
-    Param = 0;
-    Mark  = 0;
-
-    /* Check if this is a function pointer. If so, save it. If not, check for
-     * special known library functions that may be inlined.
+    ExprDesc lval;
+
+    /* Initialize variables */
+    SymEntry* Param              = 0;  /* Keep gcc silent */
+    unsigned  ParamSize   = 0; /* Size of parameters pushed */
+    unsigned  ParamCount  = 0; /* Number of parameters pushed */
+    unsigned  FrameSize   = 0; /* Size of parameter frame */
+    unsigned  FrameParams = 0; /* Number of params in frame */
+    int       FrameOffs   = 0; /* Offset into parameter frame */
+    int              Ellipsis    = 0;  /* Function is variadic */
+
+    /* As an optimization, we may allocate the complete parameter frame at
+     * once instead of pushing each parameter as it comes. We may do that,
+     * if...
+     *
+     *  - optimizations that increase code size are enabled (allocating the
+     *    stack frame at once gives usually larger code).
+     *  - we have more than one parameter to push (don't count the last param
+     *    for __fastcall__ functions).
      */
-    if (lval->e_flags & E_MEXPR) {
-               /* Function pointer is in primary register, save it */
-               Mark = GetCodePos ();
-               g_save (CF_PTR);
-    } else if (InlineStdFuncs && IsStdFunc ((const char*) lval->e_name)) {
-               /* Inline this function */
-               HandleStdFunc (lval);
-               return;
+    if (CodeSizeFactor >= 200) {
+
+       /* Calculate the number and size of the parameters */
+       FrameParams = Func->ParamCount;
+       FrameSize   = Func->ParamSize;
+       if (FrameParams > 0 && (Func->Flags & FD_FASTCALL) != 0) {
+           /* Last parameter is not pushed */
+           const SymEntry* LastParam = Func->SymTab->SymTail;
+           FrameSize -= CheckedSizeOf (LastParam->Type);
+           --FrameParams;
+       }
+
+       /* Do we have more than one parameter in the frame? */
+       if (FrameParams > 1) {
+           /* Okeydokey, setup the frame */
+           FrameOffs = oursp;
+           g_space (FrameSize);
+           oursp -= FrameSize;
+       } else {
+           /* Don't use a preallocated frame */
+           FrameSize = 0;
+       }
     }
 
     /* Parse the actual parameter list */
-    ParamSize  = 0;
-    ParamCount = 0;
-    Ellipsis   = 0;
-    while (curtok != TOK_RPAREN) {
+    while (CurTok.Tok != TOK_RPAREN) {
 
-       /* Add a hint for the optimizer */
-       AddCodeHint ("param:start");
+       unsigned CFlags;
+       unsigned Flags;
 
-       /* Count arguments */
-       ++ParamCount;
+       /* Count arguments */
+       ++ParamCount;
 
        /* Fetch the pointer to the next argument, check for too many args */
        if (ParamCount <= Func->ParamCount) {
@@ -579,7 +629,7 @@ static void callfunction (struct expent* lval)
            }
        } else if (!Ellipsis) {
            /* Too many arguments. Do we have an open param list? */
-           if ((Func->Flags & FD_ELLIPSIS) == 0) {
+           if ((Func->Flags & FD_VARIADIC) == 0) {
                /* End of param list reached, no ellipsis */
                Error ("Too many arguments in function call");
            }
@@ -593,11 +643,11 @@ static void callfunction (struct expent* lval)
         * use a special function that may optimize.
         */
                CFlags = CF_NONE;
-               if (!Ellipsis && SizeOf (Param->Type) == 1) {
+               if (!Ellipsis && CheckedSizeOf (Param->Type) == 1) {
            CFlags = CF_FORCECHAR;
        }
-       Flags = 0;
-               if (evalexpr (CFlags, hie1, &lval2) == 0) {
+       Flags = CF_NONE;
+               if (evalexpr (CFlags, hie1, &lval) == 0) {
                    /* A constant value */
            Flags |= CF_CONST;
        }
@@ -607,14 +657,14 @@ static void callfunction (struct expent* lval)
         */
                if (!Ellipsis) {
            /* Promote the argument if needed */
-                   assignadjust (Param->Type, &lval2);
+                   assignadjust (Param->Type, &lval);
 
            /* If we have a prototype, chars may be pushed as chars */
            Flags |= CF_FORCECHAR;
                }
 
        /* Use the type of the argument for the push */
-               Flags |= TypeOf (lval2.e_tptr);
+               Flags |= TypeOf (lval.Type);
 
        /* If this is a fastcall function, don't push the last argument */
                if (ParamCount == Func->ParamCount && (Func->Flags & FD_FASTCALL) != 0) {
@@ -623,98 +673,185 @@ static void callfunction (struct expent* lval)
             * the primary.
             */
            if (Flags & CF_CONST) {
-               exprhs (CF_FORCECHAR, 0, &lval2);
+               exprhs (CF_FORCECHAR, 0, &lval);
            }
        } else {
-           /* Push the argument, count the argument size */
-           g_push (Flags, lval2.e_const);
-           ParamSize += sizeofarg (Flags);
-       }
+           unsigned ArgSize = sizeofarg (Flags);
+           if (FrameSize > 0) {
+               /* We have the space already allocated, store in the frame */
+               CHECK (FrameSize >= ArgSize);
+               FrameSize -= ArgSize;
+               FrameOffs -= ArgSize;
+               /* Store */
+               g_putlocal (Flags | CF_NOKEEP, FrameOffs, lval.ConstVal);
+           } else {
+               /* Push the argument */
+               g_push (Flags, lval.ConstVal);
+           }
 
-       /* Add an optimizer hint */
-       AddCodeHint ("param:end");
+           /* Calculate total parameter size */
+           ParamSize += ArgSize;
+       }
 
        /* Check for end of argument list */
-       if (curtok != TOK_COMMA) {
+       if (CurTok.Tok != TOK_COMMA) {
            break;
        }
        NextToken ();
     }
 
-    /* We need the closing bracket here */
-    ConsumeRParen ();
-
     /* Check if we had enough parameters */
     if (ParamCount < Func->ParamCount) {
        Error ("Too few arguments in function call");
     }
 
-    /* */
-    if (lval->e_flags & E_MEXPR) {
-       /* Function called via pointer: Restore it and call function */
-       if (ParamSize != 0) {
-           g_restore (CF_PTR);
-       } else {
-           /* We had no parameters - remove save code */
-                   RemoveCode (Mark);
-       }
-       g_callind (TypeOf (lval->e_tptr), ParamSize);
-    } else {
-               g_call (TypeOf (lval->e_tptr), (char*) lval->e_name, ParamSize);
-    }
+    /* The function returns the size of all parameters pushed onto the stack.
+     * However, if there are parameters missing (which is an error and was
+     * flagged by the compiler) AND a stack frame was preallocated above,
+     * we would loose track of the stackpointer and generate an internal error
+     * later. So we correct the value by the parameters that should have been
+     * pushed to avoid an internal compiler error. Since an error was
+     * generated before, no code will be output anyway.
+     */
+    return ParamSize + FrameSize;
 }
 
 
 
-void doasm (void)
-/* This function parses ASM statements. The syntax of the ASM directive
- * looks like the one defined for C++ (C has no ASM directive), that is,
- * a string literal in parenthesis.
- */
+static void FunctionCall (int k, ExprDesc* lval)
+/* Perform a function call. */
 {
-    /* Skip the ASM */
-    NextToken ();
+    FuncDesc*    Func;           /* Function descriptor */
+    int           IsFuncPtr;      /* Flag */
+    unsigned     ParamSize;      /* Number of parameter bytes */
+    CodeMark     Mark = 0;       /* Initialize to keep gcc silent */
+    int           PtrOffs = 0;    /* Offset of function pointer on stack */
+    int           IsFastCall = 0; /* True if it's a fast call function */
+    int           PtrOnStack = 0; /* True if a pointer copy is on stack */
 
-    /* Need left parenthesis */
-    ConsumeLParen ();
+    /* Get a pointer to the function descriptor from the type string */
+    Func = GetFuncDesc (lval->Type);
+
+    /* Handle function pointers transparently */
+    IsFuncPtr = IsTypeFuncPtr (lval->Type);
+    if (IsFuncPtr) {
+
+       /* Check wether it's a fastcall function that has parameters */
+       IsFastCall = IsFastCallFunc (lval->Type + 1) && (Func->ParamCount > 0);
+
+       /* Things may be difficult, depending on where the function pointer
+        * resides. If the function pointer is an expression of some sort
+        * (not a local or global variable), we have to evaluate this
+        * expression now and save the result for later. Since calls to
+        * function pointers may be nested, we must save it onto the stack.
+        * For fastcall functions we do also need to place a copy of the
+        * pointer on stack, since we cannot use a/x.
+        */
+       PtrOnStack = IsFastCall || ((lval->Flags & (E_MGLOBAL | E_MLOCAL)) == 0);
+       if (PtrOnStack) {
 
-    /* String literal */
-    if (curtok != TOK_SCONST) {
-       Error ("String literal expected");
-    } else {
-       /* Write the string directly into the output, followed by a newline */
-               AddCodeLine (GetLiteral (curval));
+           /* Not a global or local variable, or a fastcall function. Load
+            * the pointer into the primary and mark it as an expression.
+            */
+           exprhs (CF_NONE, k, lval);
+           lval->Flags |= E_MEXPR;
+
+           /* Remember the code position */
+           Mark = GetCodePos ();
+
+           /* Push the pointer onto the stack and remember the offset */
+           g_push (CF_PTR, 0);
+           PtrOffs = oursp;
+       }
+
+    /* Check for known standard functions and inline them if requested */
+    } else if (InlineStdFuncs && IsStdFunc ((const char*) lval->Name)) {
+
+       /* Inline this function */
+               HandleStdFunc (Func, lval);
+               return;
 
-       /* Reset the string pointer, effectivly clearing the string from the
-        * string table. Since we're working with one token lookahead, this
-        * will fail if the next token is also a string token, but that's a
-        * syntax error anyway, because we expect a right paren.
-        */
-       ResetLiteralOffs (curval);
     }
 
-    /* Skip the string token */
-    NextToken ();
+    /* Parse the parameter list */
+    ParamSize = FunctionParamList (Func);
 
-    /* Closing paren needed */
+    /* We need the closing paren here */
     ConsumeRParen ();
+
+    /* Special handling for function pointers */
+    if (IsFuncPtr) {
+
+       /* If the function is not a fastcall function, load the pointer to
+        * the function into the primary.
+        */
+       if (!IsFastCall) {
+
+           /* Not a fastcall function - we may use the primary */
+                   if (PtrOnStack) {
+               /* If we have no parameters, the pointer is still in the
+                * primary. Remove the code to push it and correct the
+                * stack pointer.
+                */
+               if (ParamSize == 0) {
+                   RemoveCode (Mark);
+                   pop (CF_PTR);
+                   PtrOnStack = 0;
+               } else {
+                   /* Load from the saved copy */
+                   g_getlocal (CF_PTR, PtrOffs);
+               }
+           } else {
+               /* Load from original location */
+               exprhs (CF_NONE, k, lval);
+           }
+
+           /* Call the function */
+           g_callind (TypeOf (lval->Type+1), ParamSize, PtrOffs);
+
+       } else {
+
+           /* Fastcall function. We cannot use the primary for the function
+            * pointer and must therefore use an offset to the stack location.
+            * Since fastcall functions may never be variadic, we can use the
+            * index register for this purpose.
+            */
+           g_callind (CF_LOCAL, ParamSize, PtrOffs);
+       }
+
+       /* If we have a pointer on stack, remove it */
+       if (PtrOnStack) {
+           g_space (- (int) sizeofarg (CF_PTR));
+           pop (CF_PTR);
+       }
+
+       /* Skip T_PTR */
+       ++lval->Type;
+
+    } else {
+
+       /* Normal function */
+               g_call (TypeOf (lval->Type), (const char*) lval->Name, ParamSize);
+
+    }
 }
 
 
 
-static int primary (struct expent* lval)
+static int primary (ExprDesc* lval)
 /* This is the lowest level of the expression parser. */
 {
     int k;
 
-    /* not a test at all, yet */
-    lval->e_test = 0;
+    /* Initialize fields in the expression stucture */
+    lval->Test = 0;             /* No test */
+    lval->Sym  = 0;             /* Symbol unknown */
 
     /* Character and integer constants. */
-    if (curtok == TOK_ICONST || curtok == TOK_CCONST) {
-       lval->e_flags = E_MCONST | E_TCONST;
-       lval->e_tptr  = curtype;
-       lval->e_const = curval;
+    if (CurTok.Tok == TOK_ICONST || CurTok.Tok == TOK_CCONST) {
+       lval->Flags = E_MCONST | E_TCONST;
+       lval->Type  = CurTok.Type;
+       lval->ConstVal = CurTok.IVal;
        NextToken ();
        return 0;
     }
@@ -722,33 +859,40 @@ static int primary (struct expent* lval)
     /* Process parenthesized subexpression by calling the whole parser
      * recursively.
      */
-    if (curtok == TOK_LPAREN) {
+    if (CurTok.Tok == TOK_LPAREN) {
        NextToken ();
-       memset (lval, 0, sizeof (*lval));       /* Remove any attributes */
+       InitExprDesc (lval);            /* Remove any attributes */
        k = hie0 (lval);
                ConsumeRParen ();
        return k;
     }
 
+    /* If we run into an identifier in preprocessing mode, we assume that this
+     * is an undefined macro and replace it by a constant value of zero.
+     */
+    if (Preprocessing && CurTok.Tok == TOK_IDENT) {
+        MakeConstIntExpr (lval, 0);
+        return 0;
+    }
+
     /* All others may only be used if the expression evaluation is not called
      * recursively by the preprocessor.
      */
     if (Preprocessing) {
                /* Illegal expression in PP mode */
        Error ("Preprocessor expression expected");
-       lval->e_flags = E_MCONST;
-       lval->e_tptr = type_int;
+       MakeConstIntExpr (lval, 1);
        return 0;
     }
 
     /* Identifier? */
-    if (curtok == TOK_IDENT) {
+    if (CurTok.Tok == TOK_IDENT) {
 
        SymEntry* Sym;
        ident Ident;
 
        /* Get a pointer to the symbol table entry */
-               Sym = FindSym (CurTok.Ident);
+               Sym = lval->Sym = FindSym (CurTok.Ident);
 
        /* Is the symbol known? */
        if (Sym) {
@@ -757,7 +901,7 @@ static int primary (struct expent* lval)
            NextToken ();
 
            /* The expression type is the symbol type */
-           lval->e_tptr = Sym->Type;
+           lval->Type = Sym->Type;
 
            /* Check for illegal symbol types */
            CHECK ((Sym->Flags & SC_LABEL) != SC_LABEL);
@@ -765,63 +909,63 @@ static int primary (struct expent* lval)
                /* Cannot use type symbols */
                Error ("Variable identifier expected");
                /* Assume an int type to make lval valid */
-               lval->e_flags = E_MLOCAL | E_TLOFFS;
-               lval->e_tptr = type_int;
-               lval->e_const = 0;
+               lval->Flags = E_MLOCAL | E_TLOFFS;
+               lval->Type = type_int;
+               lval->ConstVal = 0;
                return 0;
            }
 
            /* Check for legal symbol types */
                    if ((Sym->Flags & SC_CONST) == SC_CONST) {
-               /* Enum or some other numeric constant */
-               lval->e_flags = E_MCONST;
-               lval->e_const = Sym->V.ConstVal;
+               /* Enum or some other numeric constant */
+               lval->Flags = E_MCONST | E_TCONST;
+               lval->ConstVal = Sym->V.ConstVal;
                return 0;
            } else if ((Sym->Flags & SC_FUNC) == SC_FUNC) {
                /* Function */
-               lval->e_flags = E_MGLOBAL | E_MCONST | E_TGLAB;
-               lval->e_name = (unsigned long) Sym->Name;
-               lval->e_const = 0;
+               lval->Flags = E_MGLOBAL | E_MCONST | E_TGLAB;
+               lval->Name = (unsigned long) Sym->Name;
+               lval->ConstVal = 0;
            } else if ((Sym->Flags & SC_AUTO) == SC_AUTO) {
                /* Local variable. If this is a parameter for a variadic
-                * function, we have to add some address calculations, and the
-                * address is not const.
-                */
-                       if ((Sym->Flags & SC_PARAM) == SC_PARAM && IsVariadic (CurrentFunc)) {
-                   /* Variadic parameter */
-                   g_leavariadic (Sym->V.Offs - GetParamSize (CurrentFunc));
-                   lval->e_flags = E_MEXPR;
-                   lval->e_const = 0;
-               } else {
-                   /* Normal parameter */
-                   lval->e_flags = E_MLOCAL | E_TLOFFS;
-                   lval->e_const = Sym->V.Offs;
-               }
+                * function, we have to add some address calculations, and the
+                * address is not const.
+                */
+                       if ((Sym->Flags & SC_PARAM) == SC_PARAM && F_IsVariadic (CurrentFunc)) {
+                   /* Variadic parameter */
+                   g_leavariadic (Sym->V.Offs - F_GetParamSize (CurrentFunc));
+                   lval->Flags = E_MEXPR;
+                   lval->ConstVal = 0;
+               } else {
+                   /* Normal parameter */
+                   lval->Flags = E_MLOCAL | E_TLOFFS;
+                   lval->ConstVal = Sym->V.Offs;
+               }
+           } else if ((Sym->Flags & SC_REGISTER) == SC_REGISTER) {
+               /* Register variable, zero page based */
+               lval->Flags = E_MGLOBAL | E_MCONST | E_TREGISTER;
+               lval->Name  = Sym->V.R.RegOffs;
+               lval->ConstVal = 0;
            } else if ((Sym->Flags & SC_STATIC) == SC_STATIC) {
                /* Static variable */
                if (Sym->Flags & (SC_EXTERN | SC_STORAGE)) {
-                   lval->e_flags = E_MGLOBAL | E_MCONST | E_TGLAB;
-                   lval->e_name = (unsigned long) Sym->Name;
+                   lval->Flags = E_MGLOBAL | E_MCONST | E_TGLAB;
+                   lval->Name = (unsigned long) Sym->Name;
                } else {
-                   lval->e_flags = E_MGLOBAL | E_MCONST | E_TLLAB;
-                   lval->e_name = Sym->V.Label;
+                   lval->Flags = E_MGLOBAL | E_MCONST | E_TLLAB;
+                   lval->Name = Sym->V.Label;
                }
-               lval->e_const = 0;
-           } else if ((Sym->Flags & SC_REGISTER) == SC_REGISTER) {
-               /* Register variable, zero page based */
-               lval->e_flags = E_MGLOBAL | E_MCONST | E_TREGISTER;
-               lval->e_name  = Sym->V.Offs;
-               lval->e_const = 0;
+               lval->ConstVal = 0;
                    } else {
                /* Local static variable */
-               lval->e_flags = E_MGLOBAL | E_MCONST | E_TLLAB;
-               lval->e_name  = Sym->V.Offs;
-               lval->e_const = 0;
+               lval->Flags = E_MGLOBAL | E_MCONST | E_TLLAB;
+               lval->Name  = Sym->V.Offs;
+               lval->ConstVal = 0;
            }
 
            /* The symbol is referenced now */
            Sym->Flags |= SC_REF;
-                   if (IsTypeFunc (lval->e_tptr) || IsTypeArray (lval->e_tptr)) {
+                   if (IsTypeFunc (lval->Type) || IsTypeArray (lval->Type)) {
                return 0;
            }
            return 1;
@@ -832,26 +976,26 @@ static int primary (struct expent* lval)
        NextToken ();
 
        /* IDENT is either an auto-declared function or an undefined variable. */
-       if (curtok == TOK_LPAREN) {
+       if (CurTok.Tok == TOK_LPAREN) {
            /* Declare a function returning int. For that purpose, prepare a
             * function signature for a function having an empty param list
             * and returning int.
             */
            Warning ("Function call without a prototype");
            Sym = AddGlobalSym (Ident, GetImplicitFuncType(), SC_EXTERN | SC_REF | SC_FUNC);
-           lval->e_tptr  = Sym->Type;
-           lval->e_flags = E_MGLOBAL | E_MCONST | E_TGLAB;
-                   lval->e_name  = (unsigned long) Sym->Name;
-           lval->e_const = 0;
+           lval->Type  = Sym->Type;
+           lval->Flags = E_MGLOBAL | E_MCONST | E_TGLAB;
+                   lval->Name  = (unsigned long) Sym->Name;
+           lval->ConstVal = 0;
            return 0;
 
        } else {
 
            /* Undeclared Variable */
            Sym = AddLocalSym (Ident, type_int, SC_AUTO | SC_REF, 0);
-           lval->e_flags = E_MLOCAL | E_TLOFFS;
-           lval->e_tptr = type_int;
-           lval->e_const = 0;
+           lval->Flags = E_MLOCAL | E_TLOFFS;
+           lval->Type = type_int;
+           lval->ConstVal = 0;
            Error ("Undefined symbol: `%s'", Ident);
            return 1;
 
@@ -859,43 +1003,42 @@ static int primary (struct expent* lval)
     }
 
     /* String literal? */
-    if (curtok == TOK_SCONST) {
-       lval->e_flags = E_MCONST | E_TLIT;
-               lval->e_const = curval;
-       lval->e_tptr  = GetCharArrayType (strlen (GetLiteral (curval)));
+    if (CurTok.Tok == TOK_SCONST) {
+       lval->Flags = E_MCONST | E_TLIT;
+               lval->ConstVal = CurTok.IVal;
+       lval->Type  = GetCharArrayType (GetLiteralPoolOffs () - CurTok.IVal);
        NextToken ();
        return 0;
     }
 
     /* ASM statement? */
-    if (curtok == TOK_ASM) {
-       doasm ();
-       lval->e_tptr  = type_void;
-       lval->e_flags = E_MEXPR;
-       lval->e_const = 0;
+    if (CurTok.Tok == TOK_ASM) {
+       AsmStatement ();
+       lval->Type  = type_void;
+       lval->Flags = E_MEXPR;
+       lval->ConstVal = 0;
        return 0;
     }
 
     /* __AX__ and __EAX__ pseudo values? */
-    if (curtok == TOK_AX || curtok == TOK_EAX) {
-               lval->e_tptr  = (curtok == TOK_AX)? type_uint : type_ulong;
-       lval->e_flags = E_MREG;
-       lval->e_test &= ~E_CC;
-       lval->e_const = 0;
+    if (CurTok.Tok == TOK_AX || CurTok.Tok == TOK_EAX) {
+               lval->Type  = (CurTok.Tok == TOK_AX)? type_uint : type_ulong;
+       lval->Flags = E_MREG;
+       lval->Test &= ~E_CC;
+       lval->ConstVal = 0;
        NextToken ();
        return 1;               /* May be used as lvalue */
     }
 
     /* Illegal primary. */
     Error ("Expression expected");
-    lval->e_flags = E_MCONST;
-    lval->e_tptr = type_int;
+    MakeConstIntExpr (lval, 1);
     return 0;
 }
 
 
 
-static int arrayref (int k, struct expent* lval)
+static int arrayref (int k, ExprDesc* lval)
 /* Handle an array reference */
 {
     unsigned lflags;
@@ -903,7 +1046,7 @@ static int arrayref (int k, struct expent* lval)
     int ConstBaseAddr;
     int ConstSubAddr;
     int l;
-    struct expent lval2;
+    ExprDesc lval2;
     CodeMark Mark1;
     CodeMark Mark2;
     type* tptr1;
@@ -914,13 +1057,13 @@ static int arrayref (int k, struct expent* lval)
     NextToken ();
 
     /* Get the type of left side */
-    tptr1 = lval->e_tptr;
+    tptr1 = lval->Type;
 
     /* We can apply a special treatment for arrays that have a const base
      * address. This is true for most arrays and will produce a lot better
      * code. Check if this is a const base address.
      */
-    lflags = lval->e_flags & ~E_MCTYPE;
+    lflags = lval->Flags & ~E_MCTYPE;
     ConstBaseAddr = (lflags == E_MCONST)       || /* Constant numeric address */
                             (lflags & E_MGLOBAL) != 0 || /* Static array, or ... */
                             lflags == E_MLOCAL;          /* Local array */
@@ -942,7 +1085,7 @@ static int arrayref (int k, struct expent* lval)
 
     /* TOS now contains ptr to array elements. Get the subscript. */
     l = hie0 (&lval2);
-    if (l == 0 && lval2.e_flags == E_MCONST) {
+    if (l == 0 && lval2.Flags == E_MCONST) {
 
        /* The array subscript is a constant - remove value from stack */
        if (!ConstBaseAddr) {
@@ -956,44 +1099,45 @@ static int arrayref (int k, struct expent* lval)
        if (IsClassPtr (tptr1)) {
 
            /* Scale the subscript value according to element size */
-           lval2.e_const *= PSizeOf (tptr1);
+           lval2.ConstVal *= CheckedPSizeOf (tptr1);
 
            /* Remove code for lhs load */
            RemoveCode (Mark1);
 
            /* Handle constant base array on stack. Be sure NOT to
-            * handle pointers the same way, this won't work.
+            * handle pointers the same way, and check for character literals
+             * (both won't work).
             */
-           if (IsTypeArray (tptr1) &&
-               ((lval->e_flags & ~E_MCTYPE) == E_MCONST ||
-               (lval->e_flags & ~E_MCTYPE) == E_MLOCAL ||
-               (lval->e_flags & E_MGLOBAL) != 0 ||
-               (lval->e_flags == E_MEOFFS))) {
-               lval->e_const += lval2.e_const;
+           if (IsTypeArray (tptr1) && lval->Flags != (E_MCONST | E_TLIT) &&
+               ((lval->Flags & ~E_MCTYPE) == E_MCONST ||
+               (lval->Flags & ~E_MCTYPE) == E_MLOCAL ||
+               (lval->Flags & E_MGLOBAL) != 0 ||
+               (lval->Flags == E_MEOFFS))) {
+               lval->ConstVal += lval2.ConstVal;
 
            } else {
                /* Pointer - load into primary and remember offset */
-               if ((lval->e_flags & E_MEXPR) == 0 || k != 0) {
+               if ((lval->Flags & E_MEXPR) == 0 || k != 0) {
                    exprhs (CF_NONE, k, lval);
                }
-               lval->e_const = lval2.e_const;
-               lval->e_flags = E_MEOFFS;
+               lval->ConstVal = lval2.ConstVal;
+               lval->Flags = E_MEOFFS;
            }
 
                    /* Result is of element type */
-           lval->e_tptr = Indirect (tptr1);
+           lval->Type = Indirect (tptr1);
 
            /* Done */
            goto end_array;
 
-               } else if (IsClassPtr (tptr2 = lval2.e_tptr)) {
+               } else if (IsClassPtr (tptr2 = lval2.Type)) {
            /* Subscript is pointer, get element type */
-           lval2.e_tptr = Indirect (tptr2);
+           lval2.Type = Indirect (tptr2);
 
            /* Scale the rhs value in the primary register */
-           g_scale (TypeOf (tptr1), SizeOf (lval2.e_tptr));
+           g_scale (TypeOf (tptr1), CheckedSizeOf (lval2.Type));
            /* */
-           lval->e_tptr = lval2.e_tptr;
+           lval->Type = lval2.Type;
        } else {
            Error ("Cannot subscript");
        }
@@ -1002,7 +1146,7 @@ static int arrayref (int k, struct expent* lval)
         * we will ignore the true type of the subscript here and
         * use always an int.
         */
-       g_inc (CF_INT | CF_CONST, lval2.e_const);
+       g_inc (CF_INT | CF_CONST, lval2.ConstVal);
 
     } else {
 
@@ -1010,22 +1154,22 @@ static int arrayref (int k, struct expent* lval)
        Mark2 = GetCodePos ();
         exprhs (CF_NONE, l, &lval2);
 
-       tptr2 = lval2.e_tptr;
+       tptr2 = lval2.Type;
        if (IsClassPtr (tptr1)) {
 
            /* Get the element type */
-           lval->e_tptr = Indirect (tptr1);
+           lval->Type = Indirect (tptr1);
 
                    /* Indexing is based on int's, so we will just use the integer
             * portion of the index (which is in (e)ax, so there's no further
             * action required).
             */
-           g_scale (CF_INT, SizeOf (lval->e_tptr));
+           g_scale (CF_INT, CheckedSizeOf (lval->Type));
 
        } else if (IsClassPtr (tptr2)) {
 
            /* Get the element type */
-           lval2.e_tptr = Indirect (tptr2);
+           lval2.Type = Indirect (tptr2);
 
            /* Get the int value on top. If we go here, we're sure,
             * both values are 16 bit (the first one was truncated
@@ -1042,8 +1186,8 @@ static int arrayref (int k, struct expent* lval)
            }
 
            /* Scale it */
-           g_scale (TypeOf (tptr1), SizeOf (lval2.e_tptr));
-           lval->e_tptr = lval2.e_tptr;
+           g_scale (TypeOf (tptr1), CheckedSizeOf (lval2.Type));
+           lval->Type = lval2.Type;
        } else {
            Error ("Cannot subscript");
        }
@@ -1066,69 +1210,69 @@ static int arrayref (int k, struct expent* lval)
             * subscript was not scaled, that is, if this was a byte array
             * or pointer.
             */
-           rflags = lval2.e_flags & ~E_MCTYPE;
+           rflags = lval2.Flags & ~E_MCTYPE;
            ConstSubAddr = (rflags == E_MCONST)       || /* Constant numeric address */
                                    (rflags & E_MGLOBAL) != 0 || /* Static array, or ... */
                            rflags == E_MLOCAL;          /* Local array */
 
-                   if (ConstSubAddr && SizeOf (lval->e_tptr) == 1) {
+                   if (ConstSubAddr && CheckedSizeOf (lval->Type) == SIZEOF_CHAR) {
 
                type* SavedType;
 
                /* Reverse the order of evaluation */
-               unsigned flags = (SizeOf (lval2.e_tptr) == 1)? CF_CHAR : CF_INT;
+               unsigned flags = (CheckedSizeOf (lval2.Type) == SIZEOF_CHAR)? CF_CHAR : CF_INT;
                RemoveCode (Mark2);
 
                /* Get a pointer to the array into the primary. We have changed
-                * e_tptr above but we need the original type to load the
+                * Type above but we need the original type to load the
                 * address, so restore it temporarily.
                 */
-               SavedType = lval->e_tptr;
-               lval->e_tptr = tptr1;
+               SavedType = lval->Type;
+               lval->Type = tptr1;
                exprhs (CF_NONE, k, lval);
-               lval->e_tptr = SavedType;
+               lval->Type = SavedType;
 
                /* Add the variable */
                if (rflags == E_MLOCAL) {
-                   g_addlocal (flags, lval2.e_const);
+                   g_addlocal (flags, lval2.ConstVal);
                } else {
-                   flags |= GlobalModeFlags (lval2.e_flags);
-                   g_addstatic (flags, lval2.e_name, lval2.e_const);
+                   flags |= GlobalModeFlags (lval2.Flags);
+                   g_addstatic (flags, lval2.Name, lval2.ConstVal);
                }
            } else {
                if (lflags == E_MCONST) {
                    /* Constant numeric address. Just add it */
-                   g_inc (CF_INT | CF_UNSIGNED, lval->e_const);
+                   g_inc (CF_INT | CF_UNSIGNED, lval->ConstVal);
                } else if (lflags == E_MLOCAL) {
-                   /* Base address is a local variable address */
-                   if (IsTypeArray (tptr1)) {
-                       g_addaddr_local (CF_INT, lval->e_const);
-                   } else {
-                       g_addlocal (CF_PTR, lval->e_const);
-                   }
+                   /* Base address is a local variable address */
+                   if (IsTypeArray (tptr1)) {
+                       g_addaddr_local (CF_INT, lval->ConstVal);
+                   } else {
+                       g_addlocal (CF_PTR, lval->ConstVal);
+                   }
                } else {
                    /* Base address is a static variable address */
                    unsigned flags = CF_INT;
-                   flags |= GlobalModeFlags (lval->e_flags);
-                   if (IsTypeArray (tptr1)) {
-                       g_addaddr_static (flags, lval->e_name, lval->e_const);
-                   } else {
-                       g_addstatic (flags, lval->e_name, lval->e_const);
-                   }
+                   flags |= GlobalModeFlags (lval->Flags);
+                   if (IsTypeArray (tptr1)) {
+                       g_addaddr_static (flags, lval->Name, lval->ConstVal);
+                   } else {
+                       g_addstatic (flags, lval->Name, lval->ConstVal);
+                   }
                }
            }
        }
     }
-    lval->e_flags = E_MEXPR;
+    lval->Flags = E_MEXPR;
 end_array:
     ConsumeRBrack ();
-    return !IsTypeArray (lval->e_tptr);
+    return !IsTypeArray (lval->Type);
 
 }
 
 
 
-static int structref (int k, struct expent* lval)
+static int structref (int k, ExprDesc* lval)
 /* Process struct field after . or ->. */
 {
     ident Ident;
@@ -1137,43 +1281,43 @@ static int structref (int k, struct expent* lval)
 
     /* Skip the token and check for an identifier */
     NextToken ();
-    if (curtok != TOK_IDENT) {
+    if (CurTok.Tok != TOK_IDENT) {
        Error ("Identifier expected");
-       lval->e_tptr = type_int;
+       lval->Type = type_int;
        return 0;
     }
 
     /* Get the symbol table entry and check for a struct field */
     strcpy (Ident, CurTok.Ident);
     NextToken ();
-    Field = FindStructField (lval->e_tptr, Ident);
+    Field = FindStructField (lval->Type, Ident);
     if (Field == 0) {
        Error ("Struct/union has no field named `%s'", Ident);
-               lval->e_tptr = type_int;
+               lval->Type = type_int;
        return 0;
     }
 
     /* If we have constant input data, the result is also constant */
-    flags = lval->e_flags & ~E_MCTYPE;
+    flags = lval->Flags & ~E_MCTYPE;
     if (flags == E_MCONST ||
                (k == 0 && (flags == E_MLOCAL ||
-                   (flags & E_MGLOBAL) != 0 ||
-                   lval->e_flags  == E_MEOFFS))) {
-       lval->e_const += Field->V.Offs;
+                   (flags & E_MGLOBAL) != 0 ||
+                   lval->Flags  == E_MEOFFS))) {
+       lval->ConstVal += Field->V.Offs;
     } else {
        if ((flags & E_MEXPR) == 0 || k != 0) {
            exprhs (CF_NONE, k, lval);
        }
-       lval->e_const = Field->V.Offs;
-       lval->e_flags = E_MEOFFS;
+       lval->ConstVal = Field->V.Offs;
+       lval->Flags = E_MEOFFS;
     }
-    lval->e_tptr = Field->Type;
+    lval->Type = Field->Type;
     return !IsTypeArray (Field->Type);
 }
 
 
 
-static int hie11 (struct expent *lval)
+static int hie11 (ExprDesc *lval)
 /* Handle compound types (structs and arrays) */
 {
     int k;
@@ -1181,48 +1325,49 @@ static int hie11 (struct expent *lval)
 
 
     k = primary (lval);
-    if (curtok < TOK_LBRACK || curtok > TOK_PTR_REF) {
+    if (CurTok.Tok < TOK_LBRACK || CurTok.Tok > TOK_PTR_REF) {
        /* Not for us */
                return k;
     }
 
     while (1) {
 
-       if (curtok == TOK_LBRACK) {
+       if (CurTok.Tok == TOK_LBRACK) {
 
            /* Array reference */
            k = arrayref (k, lval);
 
-       } else if (curtok == TOK_LPAREN) {
+       } else if (CurTok.Tok == TOK_LPAREN) {
 
            /* Function call. Skip the opening parenthesis */
            NextToken ();
-           tptr = lval->e_tptr;
-           if (IsTypeFunc (tptr) || IsTypeFuncPtr (tptr)) {
-               if (IsTypeFuncPtr (tptr)) {
-                   /* Pointer to function. Handle transparently */
-                   exprhs (CF_NONE, k, lval);  /* Function pointer to A/X */
-                   ++lval->e_tptr;             /* Skip T_PTR */
-                   lval->e_flags |= E_MEXPR;
-               }
-               callfunction (lval);
-               lval->e_flags = E_MEXPR;
-               lval->e_tptr += DECODE_SIZE + 1;        /* Set to result */
+           tptr = lval->Type;
+           if (IsTypeFunc (lval->Type) || IsTypeFuncPtr (lval->Type)) {
+
+               /* Call the function */
+               FunctionCall (k, lval);
+
+               /* Result is in the primary register */
+               lval->Flags = E_MEXPR;
+
+               /* Set to result */
+               lval->Type = GetFuncReturn (lval->Type);
+
            } else {
                Error ("Illegal function call");
            }
            k = 0;
 
-       } else if (curtok == TOK_DOT) {
+       } else if (CurTok.Tok == TOK_DOT) {
 
-           if (!IsClassStruct (lval->e_tptr)) {
+           if (!IsClassStruct (lval->Type)) {
                Error ("Struct expected");
            }
            k = structref (0, lval);
 
-       } else if (curtok == TOK_PTR_REF) {
+       } else if (CurTok.Tok == TOK_PTR_REF) {
 
-           tptr = lval->e_tptr;
+           tptr = lval->Type;
            if (tptr[0] != T_PTR || (tptr[1] & T_STRUCT) == 0) {
                Error ("Struct pointer expected");
            }
@@ -1236,44 +1381,56 @@ static int hie11 (struct expent *lval)
 
 
 
-static void store (struct expent* lval)
-/* Store primary reg into this reference */
+void Store (ExprDesc* lval, const type* StoreType)
+/* Store the primary register into the location denoted by lval. If StoreType
+ * is given, use this type when storing instead of lval->Type. If StoreType
+ * is NULL, use lval->Type instead.
+ */
 {
-    int f;
-    unsigned flags;
+    unsigned Flags;
+
+    unsigned f = lval->Flags;
 
-    f = lval->e_flags;
-    flags = TypeOf (lval->e_tptr);
+    /* If StoreType was not given, use lval->Type instead */
+    if (StoreType == 0) {
+        StoreType = lval->Type;
+    }
+
+    /* Get the code generator flags */
+    Flags = TypeOf (StoreType);
     if (f & E_MGLOBAL) {
-       flags |= GlobalModeFlags (f);
-       if (lval->e_test) {
-           /* Just testing */
-                   flags |= CF_TEST;
-       }
+       Flags |= GlobalModeFlags (f);
+       if (lval->Test) {
+           /* Just testing */
+                   Flags |= CF_TEST;
+       }
 
        /* Generate code */
-               g_putstatic (flags, lval->e_name, lval->e_const);
+               g_putstatic (Flags, lval->Name, lval->ConstVal);
 
     } else if (f & E_MLOCAL) {
-               g_putlocal (flags, lval->e_const);
+        /* Store an auto variable */
+               g_putlocal (Flags, lval->ConstVal, 0);
     } else if (f == E_MEOFFS) {
-       g_putind (flags, lval->e_const);
+        /* Store indirect with offset */
+       g_putind (Flags, lval->ConstVal);
     } else if (f != E_MREG) {
        if (f & E_MEXPR) {
-           g_putind (flags, 0);
+            /* Indirect without offset */
+           g_putind (Flags, 0);
        } else {
            /* Store into absolute address */
-           g_putstatic (flags | CF_ABSOLUTE, lval->e_const, 0);
+           g_putstatic (Flags | CF_ABSOLUTE, lval->ConstVal, 0);
        }
     }
 
     /* Assume that each one of the stores will invalidate CC */
-    lval->e_test &= ~E_CC;
+    lval->Test &= ~E_CC;
 }
 
 
 
-static void pre_incdec (struct expent* lval, void (*inc) (unsigned, unsigned long))
+static void pre_incdec (ExprDesc* lval, void (*inc) (unsigned, unsigned long))
 /* Handle --i and ++i */
 {
     int k;
@@ -1287,16 +1444,16 @@ static void pre_incdec (struct expent* lval, void (*inc) (unsigned, unsigned lon
     }
 
     /* Get the data type */
-    flags = TypeOf (lval->e_tptr) | CF_FORCECHAR | CF_CONST;
+    flags = TypeOf (lval->Type) | CF_FORCECHAR | CF_CONST;
 
     /* Get the increment value in bytes */
-    val = (lval->e_tptr [0] == T_PTR)? PSizeOf (lval->e_tptr) : 1;
+    val = (lval->Type [0] == T_PTR)? CheckedPSizeOf (lval->Type) : 1;
 
     /* We're currently only able to handle some adressing modes */
-    if ((lval->e_flags & E_MGLOBAL) == 0 &&    /* Global address? */
-       (lval->e_flags & E_MLOCAL) == 0  &&     /* Local address? */
-       (lval->e_flags & E_MCONST) == 0  &&     /* Constant address? */
-       (lval->e_flags & E_MEXPR) == 0) {       /* Address in a/x? */
+    if ((lval->Flags & E_MGLOBAL) == 0 &&      /* Global address? */
+       (lval->Flags & E_MLOCAL) == 0  &&       /* Local address? */
+       (lval->Flags & E_MCONST) == 0  &&       /* Constant address? */
+       (lval->Flags & E_MEXPR) == 0) {         /* Address in a/x? */
 
        /* Use generic code. Push the address if needed */
        PushAddr (lval);
@@ -1308,36 +1465,36 @@ static void pre_incdec (struct expent* lval, void (*inc) (unsigned, unsigned lon
                inc (flags, val);
 
        /* Store the result back */
-       store (lval);
+       Store (lval, 0);
 
     } else {
 
        /* Special code for some addressing modes - use the special += ops */
-       if (lval->e_flags & E_MGLOBAL) {
-           flags |= GlobalModeFlags (lval->e_flags);
+       if (lval->Flags & E_MGLOBAL) {
+           flags |= GlobalModeFlags (lval->Flags);
            if (inc == g_inc) {
-               g_addeqstatic (flags, lval->e_name, lval->e_const, val);
+               g_addeqstatic (flags, lval->Name, lval->ConstVal, val);
            } else {
-               g_subeqstatic (flags, lval->e_name, lval->e_const, val);
+               g_subeqstatic (flags, lval->Name, lval->ConstVal, val);
            }
-       } else if (lval->e_flags & E_MLOCAL) {
+       } else if (lval->Flags & E_MLOCAL) {
            /* ref to localvar */
            if (inc == g_inc) {
-               g_addeqlocal (flags, lval->e_const, val);
+               g_addeqlocal (flags, lval->ConstVal, val);
            } else {
-               g_subeqlocal (flags, lval->e_const, val);
+               g_subeqlocal (flags, lval->ConstVal, val);
            }
-       } else if (lval->e_flags & E_MCONST) {
+       } else if (lval->Flags & E_MCONST) {
            /* ref to absolute address */
            flags |= CF_ABSOLUTE;
            if (inc == g_inc) {
-               g_addeqstatic (flags, lval->e_const, 0, val);
+               g_addeqstatic (flags, lval->ConstVal, 0, val);
            } else {
-               g_subeqstatic (flags, lval->e_const, 0, val);
+               g_subeqstatic (flags, lval->ConstVal, 0, val);
            }
-       } else if (lval->e_flags & E_MEXPR) {
+       } else if (lval->Flags & E_MEXPR) {
            /* Address in a/x, check if we have an offset */
-           unsigned Offs = (lval->e_flags == E_MEOFFS)? lval->e_const : 0;
+           unsigned Offs = (lval->Flags == E_MEOFFS)? lval->ConstVal : 0;
            if (inc == g_inc) {
                g_addeqind (flags, Offs, val);
            } else {
@@ -1350,12 +1507,12 @@ static void pre_incdec (struct expent* lval, void (*inc) (unsigned, unsigned lon
     }
 
     /* Result is an expression */
-    lval->e_flags = E_MEXPR;
+    lval->Flags = E_MEXPR;
 }
 
 
 
-static void post_incdec (struct expent *lval, int k, void (*inc) (unsigned, unsigned long))
+static void post_incdec (ExprDesc* lval, int k, void (*inc) (unsigned, unsigned long))
 /* Handle i-- and i++ */
 {
     unsigned flags;
@@ -1367,7 +1524,7 @@ static void post_incdec (struct expent *lval, int k, void (*inc) (unsigned, unsi
     }
 
     /* Get the data type */
-    flags = TypeOf (lval->e_tptr);
+    flags = TypeOf (lval->Type);
 
     /* Push the address if needed */
     PushAddr (lval);
@@ -1377,23 +1534,23 @@ static void post_incdec (struct expent *lval, int k, void (*inc) (unsigned, unsi
     g_save (flags | CF_FORCECHAR);
 
     /* If we have a pointer expression, increment by the size of the type */
-    if (lval->e_tptr[0] == T_PTR) {
-       inc (flags | CF_CONST | CF_FORCECHAR, SizeOf (lval->e_tptr + 1));
+    if (lval->Type[0] == T_PTR) {
+       inc (flags | CF_CONST | CF_FORCECHAR, CheckedSizeOf (lval->Type + 1));
     } else {
        inc (flags | CF_CONST | CF_FORCECHAR, 1);
     }
 
     /* Store the result back */
-    store (lval);
+    Store (lval, 0);
 
     /* Restore the original value */
     g_restore (flags | CF_FORCECHAR);
-    lval->e_flags = E_MEXPR;
+    lval->Flags = E_MEXPR;
 }
 
 
 
-static void unaryop (int tok, struct expent* lval)
+static void unaryop (int tok, ExprDesc* lval)
 /* Handle unary -/+ and ~ */
 {
     int k;
@@ -1401,12 +1558,12 @@ static void unaryop (int tok, struct expent* lval)
 
     NextToken ();
     k = hie10 (lval);
-    if (k == 0 && lval->e_flags & E_MCONST) {
+    if (k == 0 && (lval->Flags & E_MCONST) != 0) {
        /* Value is constant */
        switch (tok) {
-           case TOK_MINUS: lval->e_const =     -lval->e_const; break;
+           case TOK_MINUS: lval->ConstVal = -lval->ConstVal;   break;
            case TOK_PLUS:                                      break;
-           case TOK_COMP:  lval->e_const = ~lval->e_const;     break;
+           case TOK_COMP:  lval->ConstVal = ~lval->ConstVal;   break;
            default:        Internal ("Unexpected token: %d", tok);
        }
     } else {
@@ -1414,7 +1571,7 @@ static void unaryop (int tok, struct expent* lval)
        exprhs (CF_NONE, k, lval);
 
        /* Get the type of the expression */
-       flags = TypeOf (lval->e_tptr);
+       flags = TypeOf (lval->Type);
 
        /* Handle the operation */
        switch (tok) {
@@ -1423,113 +1580,19 @@ static void unaryop (int tok, struct expent* lval)
            case TOK_COMP:  g_com (flags);  break;
            default:    Internal ("Unexpected token: %d", tok);
        }
-       lval->e_flags = E_MEXPR;
-    }
-}
-
-
-
-static int typecast (struct expent* lval)
-/* Handle an explicit cast */
-{
-    int k;
-    type Type[MAXTYPELEN];
-
-    /* Skip the left paren */
-    NextToken ();
-
-    /* Read the type */
-    ParseType (Type);
-
-    /* Closing paren */
-    ConsumeRParen ();
-
-    /* Read the expression we have to cast */
-    k = hie10 (lval);
-
-    /* If the expression is a function, treat it as pointer-to-function */
-    if (IsTypeFunc (lval->e_tptr)) {
-       lval->e_tptr = PointerTo (lval->e_tptr);
-    }
-
-    /* Check for a constant on the right side */
-    if (k == 0 && lval->e_flags == E_MCONST) {
-
-       /* A cast of a constant to something else. If the new type is an int,
-        * be sure to handle the size extension correctly. If the new type is
-        * not an int, the cast is implementation specific anyway, so leave
-        * the value alone.
-        */
-       if (IsClassInt (Type)) {
-
-           /* Get the current and new size of the value */
-           unsigned OldSize = SizeOf (lval->e_tptr);
-           unsigned NewSize = SizeOf (Type);
-           unsigned OldBits = OldSize * 8;
-           unsigned NewBits = NewSize * 8;
-
-           /* Check if the new datatype will have a smaller range */
-           if (NewSize < OldSize) {
-
-               /* Cut the value to the new size */
-               lval->e_const &= (0xFFFFFFFFUL >> (32 - NewBits));
-
-               /* If the new value is signed, sign extend the value */
-               if (!IsSignUnsigned (Type)) {
-                   lval->e_const |= ((~0L) << NewBits);
-               }
-
-           } else if (NewSize > OldSize) {
-
-               /* Sign extend the value if needed */
-               if (!IsSignUnsigned (Type) && !IsSignUnsigned (lval->e_tptr)) {
-                   if (lval->e_const & (0x01UL << (OldBits-1))) {
-                       lval->e_const |= ((~0L) << OldBits);
-                   }
-               }
-           }
-       }
-
-    } else {
-
-       /* Not a constant. Be sure to ignore casts to void */
-       if (!IsTypeVoid (Type)) {
-
-           /* If the size does not change, leave the value alone. Otherwise,
-            * we have to load the value into the primary and generate code to
-            * cast the value in the primary register.
-            */
-           if (SizeOf (Type) != SizeOf (lval->e_tptr)) {
-
-               /* Load the value into the primary */
-               exprhs (CF_NONE, k, lval);
-
-               /* Mark the lhs as const to avoid a manipulation of TOS */
-               g_typecast (TypeOf (Type) | CF_CONST, TypeOf (lval->e_tptr));
-
-               /* Value is now in primary */
-               lval->e_flags = E_MEXPR;
-                       k = 0;
-           }
-       }
+       lval->Flags = E_MEXPR;
     }
-
-    /* In any case, use the new type */
-    lval->e_tptr = TypeDup (Type);
-
-    /* Done */
-    return k;
 }
 
 
 
-static int hie10 (struct expent* lval)
+int hie10 (ExprDesc* lval)
 /* Handle ++, --, !, unary - etc. */
 {
     int k;
     type* t;
 
-    switch (curtok) {
+    switch (CurTok.Tok) {
 
        case TOK_INC:
            pre_incdec (lval, g_inc);
@@ -1542,35 +1605,43 @@ static int hie10 (struct expent* lval)
        case TOK_PLUS:
        case TOK_MINUS:
        case TOK_COMP:
-           unaryop (curtok, lval);
+           unaryop (CurTok.Tok, lval);
            return 0;
 
        case TOK_BOOL_NOT:
            NextToken ();
            if (evalexpr (CF_NONE, hie10, lval) == 0) {
-               /* Constant expression */
-               lval->e_const = !lval->e_const;
+               /* Constant expression */
+               lval->ConstVal = !lval->ConstVal;
            } else {
-               g_bneg (TypeOf (lval->e_tptr));
-               lval->e_test |= E_CC;                   /* bneg will set cc */
-               lval->e_flags = E_MEXPR;                /* say it's an expr */
+               g_bneg (TypeOf (lval->Type));
+               lval->Test |= E_CC;                     /* bneg will set cc */
+               lval->Flags = E_MEXPR;          /* say it's an expr */
            }
            return 0;                           /* expr not storable */
 
        case TOK_STAR:
            NextToken ();
            if (evalexpr (CF_NONE, hie10, lval) != 0) {
-               /* Expression is not const, indirect value loaded into primary */
-               lval->e_flags = E_MEXPR;
-               lval->e_const = 0;              /* Offset is zero now */
+               /* Expression is not const, indirect value loaded into primary */
+               lval->Flags = E_MEXPR;
+               lval->ConstVal = 0;             /* Offset is zero now */
            }
-           t = lval->e_tptr;
-                   if (IsClassPtr (t)) {
-                       lval->e_tptr = Indirect (t);
-           } else {
-               Error ("Illegal indirection");
-           }
-           return 1;
+            /* If the expression is already a pointer to function, the
+             * additional dereferencing operator must be ignored.
+             */
+            if (IsTypeFuncPtr (lval->Type)) {
+                /* Expression not storable */
+                return 0;
+            } else {
+                if (IsClassPtr (lval->Type)) {
+                    lval->Type = Indirect (lval->Type);
+                } else {
+                    Error ("Illegal indirection");
+                }
+                return 1;
+            }
+            break;
 
        case TOK_AND:
            NextToken ();
@@ -1578,48 +1649,48 @@ static int hie10 (struct expent* lval)
            /* The & operator may be applied to any lvalue, and it may be
             * applied to functions, even if they're no lvalues.
             */
-           if (k == 0 && !IsTypeFunc (lval->e_tptr)) {
-               /* Allow the & operator with an array */
-               if (!IsTypeArray (lval->e_tptr)) {
-                   Error ("Illegal address");
-               }
+           if (k == 0 && !IsTypeFunc (lval->Type)) {
+               /* Allow the & operator with an array */
+               if (!IsTypeArray (lval->Type)) {
+                   Error ("Illegal address");
+               }
            } else {
-               t = TypeAlloc (TypeLen (lval->e_tptr) + 2);
-               t [0] = T_PTR;
-               TypeCpy (t + 1, lval->e_tptr);
-               lval->e_tptr = t;
+               t = TypeAlloc (TypeLen (lval->Type) + 2);
+               t [0] = T_PTR;
+               TypeCpy (t + 1, lval->Type);
+               lval->Type = t;
            }
            return 0;
 
        case TOK_SIZEOF:
            NextToken ();
                    if (istypeexpr ()) {
-               type Type[MAXTYPELEN];
-               NextToken ();
-               lval->e_const = SizeOf (ParseType (Type));
-               ConsumeRParen ();
+               type Type[MAXTYPELEN];
+               NextToken ();
+               lval->ConstVal = CheckedSizeOf (ParseType (Type));
+               ConsumeRParen ();
            } else {
-               /* Remember the output queue pointer */
-               CodeMark Mark = GetCodePos ();
-               hie10 (lval);
-               lval->e_const = SizeOf (lval->e_tptr);
-               /* Remove any generated code */
-               RemoveCode (Mark);
+               /* Remember the output queue pointer */
+               CodeMark Mark = GetCodePos ();
+               hie10 (lval);
+               lval->ConstVal = CheckedSizeOf (lval->Type);
+               /* Remove any generated code */
+               RemoveCode (Mark);
            }
-           lval->e_flags = E_MCONST | E_TCONST;
-           lval->e_tptr = type_uint;
-           lval->e_test &= ~E_CC;
+           lval->Flags = E_MCONST | E_TCONST;
+           lval->Type = type_uint;
+           lval->Test &= ~E_CC;
            return 0;
 
        default:
                    if (istypeexpr ()) {
-               /* A cast */
-               return typecast (lval);
+               /* A cast */
+               return TypeCast (lval);
            }
     }
 
     k = hie11 (lval);
-    switch (curtok) {
+    switch (CurTok.Tok) {
        case TOK_INC:
                    post_incdec (lval, k, g_inc);
            return 0;
@@ -1635,17 +1706,17 @@ static int hie10 (struct expent* lval)
 
 
 
-static int hie_internal (GenDesc** ops,                /* List of generators */
-                                struct expent* lval,   /* parent expr's lval */
-                                int (*hienext) (struct expent*),
-                        int* UsedGen)          /* next higher level */
+static int hie_internal (const GenDesc** ops,          /* List of generators */
+                                ExprDesc* lval,        /* parent expr's lval */
+                                int (*hienext) (ExprDesc*),
+                        int* UsedGen)          /* next higher level */
 /* Helper function */
 {
     int k;
-    struct expent lval2;
+    ExprDesc lval2;
     CodeMark Mark1;
     CodeMark Mark2;
-    GenDesc* Gen;
+    const GenDesc* Gen;
     token_t tok;                               /* The operator token */
     unsigned ltype, type;
     int rconst;                                /* Operand is a constant */
@@ -1654,27 +1725,27 @@ static int hie_internal (GenDesc** ops,         /* List of generators */
     k = hienext (lval);
 
     *UsedGen = 0;
-    while ((Gen = FindGen (curtok, ops)) != 0) {
+    while ((Gen = FindGen (CurTok.Tok, ops)) != 0) {
 
        /* Tell the caller that we handled it's ops */
        *UsedGen = 1;
 
        /* All operators that call this function expect an int on the lhs */
-       if (!IsClassInt (lval->e_tptr)) {
+       if (!IsClassInt (lval->Type)) {
            Error ("Integer expression expected");
        }
 
        /* Remember the operator token, then skip it */
-               tok = curtok;
+               tok = CurTok.Tok;
        NextToken ();
 
        /* Get the lhs on stack */
                Mark1 = GetCodePos ();
-       ltype = TypeOf (lval->e_tptr);
-       if (k == 0 && lval->e_flags == E_MCONST) {
+       ltype = TypeOf (lval->Type);
+       if (k == 0 && lval->Flags == E_MCONST) {
            /* Constant value */
            Mark2 = GetCodePos ();
-                   g_push (ltype | CF_CONST, lval->e_const);
+                   g_push (ltype | CF_CONST, lval->ConstVal);
        } else {
            /* Value not constant */
            exprhs (CF_NONE, k, lval);
@@ -1686,22 +1757,22 @@ static int hie_internal (GenDesc** ops,         /* List of generators */
        rconst = (evalexpr (CF_NONE, hienext, &lval2) == 0);
 
        /* Check the type of the rhs */
-       if (!IsClassInt (lval2.e_tptr)) {
+       if (!IsClassInt (lval2.Type)) {
            Error ("Integer expression expected");
        }
 
        /* Check for const operands */
-       if (k == 0 && lval->e_flags == E_MCONST && rconst) {
+       if (k == 0 && lval->Flags == E_MCONST && rconst) {
 
            /* Both operands are constant, remove the generated code */
            RemoveCode (Mark1);
            pop (ltype);
 
            /* Evaluate the result */
-           lval->e_const = kcalc (tok, lval->e_const, lval2.e_const);
+           lval->ConstVal = kcalc (tok, lval->ConstVal, lval2.ConstVal);
 
            /* Get the type of the result */
-           lval->e_tptr = promoteint (lval->e_tptr, lval2.e_tptr);
+           lval->Type = promoteint (lval->Type, lval2.Type);
 
        } else {
 
@@ -1709,15 +1780,15 @@ static int hie_internal (GenDesc** ops,         /* List of generators */
             * expects the lhs in the primary, remove the push of the primary
             * now.
             */
-           unsigned rtype = TypeOf (lval2.e_tptr);
+           unsigned rtype = TypeOf (lval2.Type);
            type = 0;
            if (rconst) {
                /* Second value is constant - check for div */
                type |= CF_CONST;
                rtype |= CF_CONST;
-               if (tok == TOK_DIV && lval2.e_const == 0) {
+               if (tok == TOK_DIV && lval2.ConstVal == 0) {
                    Error ("Division by zero");
-               } else if (tok == TOK_MOD && lval2.e_const == 0) {
+               } else if (tok == TOK_MOD && lval2.ConstVal == 0) {
                    Error ("Modulo operation with zero");
                }
                if ((Gen->Flags & GEN_NOPUSH) != 0) {
@@ -1729,11 +1800,11 @@ static int hie_internal (GenDesc** ops,         /* List of generators */
 
            /* Determine the type of the operation result. */
                    type |= g_typeadjust (ltype, rtype);
-           lval->e_tptr = promoteint (lval->e_tptr, lval2.e_tptr);
+           lval->Type = promoteint (lval->Type, lval2.Type);
 
            /* Generate code */
-           Gen->Func (type, lval2.e_const);
-           lval->e_flags = E_MEXPR;
+           Gen->Func (type, lval2.ConstVal);
+           lval->Flags = E_MEXPR;
        }
 
        /* We have a rvalue now */
@@ -1745,16 +1816,16 @@ static int hie_internal (GenDesc** ops,         /* List of generators */
 
 
 
-static int hie_compare (GenDesc** ops,         /* List of generators */
-                               struct expent* lval,    /* parent expr's lval */
-                               int (*hienext) (struct expent*))
+static int hie_compare (const GenDesc** ops,   /* List of generators */
+                               ExprDesc* lval,         /* parent expr's lval */
+                               int (*hienext) (ExprDesc*))
 /* Helper function for the compare operators */
 {
     int k;
-    struct expent lval2;
+    ExprDesc lval2;
     CodeMark Mark1;
     CodeMark Mark2;
-    GenDesc* Gen;
+    const GenDesc* Gen;
     token_t tok;                       /* The operator token */
     unsigned ltype;
     int rconst;                                /* Operand is a constant */
@@ -1762,19 +1833,19 @@ static int hie_compare (GenDesc** ops,          /* List of generators */
 
     k = hienext (lval);
 
-    while ((Gen = FindGen (curtok, ops)) != 0) {
+    while ((Gen = FindGen (CurTok.Tok, ops)) != 0) {
 
        /* Remember the operator token, then skip it */
-               tok = curtok;
+               tok = CurTok.Tok;
        NextToken ();
 
        /* Get the lhs on stack */
        Mark1 = GetCodePos ();
-       ltype = TypeOf (lval->e_tptr);
-       if (k == 0 && lval->e_flags == E_MCONST) {
+       ltype = TypeOf (lval->Type);
+       if (k == 0 && lval->Flags == E_MCONST) {
            /* Constant value */
            Mark2 = GetCodePos ();
-                   g_push (ltype | CF_CONST, lval->e_const);
+                   g_push (ltype | CF_CONST, lval->ConstVal);
        } else {
            /* Value not constant */
            exprhs (CF_NONE, k, lval);
@@ -1786,17 +1857,17 @@ static int hie_compare (GenDesc** ops,          /* List of generators */
        rconst = (evalexpr (CF_NONE, hienext, &lval2) == 0);
 
        /* Make sure, the types are compatible */
-       if (IsClassInt (lval->e_tptr)) {
-           if (!IsClassInt (lval2.e_tptr) && !(IsClassPtr(lval2.e_tptr) && IsNullPtr(lval))) {
+       if (IsClassInt (lval->Type)) {
+           if (!IsClassInt (lval2.Type) && !(IsClassPtr(lval2.Type) && IsNullPtr(lval))) {
                Error ("Incompatible types");
            }
-       } else if (IsClassPtr (lval->e_tptr)) {
-           if (IsClassPtr (lval2.e_tptr)) {
+       } else if (IsClassPtr (lval->Type)) {
+           if (IsClassPtr (lval2.Type)) {
                /* Both pointers are allowed in comparison if they point to
                 * the same type, or if one of them is a void pointer.
                 */
-                       type* left  = Indirect (lval->e_tptr);
-               type* right = Indirect (lval2.e_tptr);
+                       type* left  = Indirect (lval->Type);
+               type* right = Indirect (lval2.Type);
                if (TypeCmp (left, right) < TC_EQUAL && *left != T_VOID && *right != T_VOID) {
                    /* Incomatible pointers */
                    Error ("Incompatible types");
@@ -1807,14 +1878,14 @@ static int hie_compare (GenDesc** ops,          /* List of generators */
        }
 
        /* Check for const operands */
-       if (k == 0 && lval->e_flags == E_MCONST && rconst) {
+       if (k == 0 && lval->Flags == E_MCONST && rconst) {
 
            /* Both operands are constant, remove the generated code */
            RemoveCode (Mark1);
            pop (ltype);
 
            /* Evaluate the result */
-           lval->e_const = kcalc (tok, lval->e_const, lval2.e_const);
+           lval->ConstVal = kcalc (tok, lval->ConstVal, lval2.ConstVal);
 
        } else {
 
@@ -1838,30 +1909,30 @@ static int hie_compare (GenDesc** ops,          /* List of generators */
             * operation as char operation. Otherwise the default
             * promotions are used.
             */
-           if (IsTypeChar (lval->e_tptr) && (IsTypeChar (lval2.e_tptr) || rconst)) {
+           if (IsTypeChar (lval->Type) && (IsTypeChar (lval2.Type) || rconst)) {
                flags |= CF_CHAR;
-               if (IsSignUnsigned (lval->e_tptr) || IsSignUnsigned (lval2.e_tptr)) {
+               if (IsSignUnsigned (lval->Type) || IsSignUnsigned (lval2.Type)) {
                    flags |= CF_UNSIGNED;
                }
                if (rconst) {
                    flags |= CF_FORCECHAR;
                }
            } else {
-               unsigned rtype = TypeOf (lval2.e_tptr) | (flags & CF_CONST);
+               unsigned rtype = TypeOf (lval2.Type) | (flags & CF_CONST);
                        flags |= g_typeadjust (ltype, rtype);
            }
 
            /* Generate code */
-           Gen->Func (flags, lval2.e_const);
-           lval->e_flags = E_MEXPR;
+           Gen->Func (flags, lval2.ConstVal);
+           lval->Flags = E_MEXPR;
        }
 
        /* Result type is always int */
-               lval->e_tptr = type_int;
+               lval->Type = type_int;
 
        /* We have a rvalue now, condition codes are set */
        k = 0;
-       lval->e_test |= E_CC;
+       lval->Test |= E_CC;
     }
 
     return k;
@@ -1869,10 +1940,10 @@ static int hie_compare (GenDesc** ops,          /* List of generators */
 
 
 
-static int hie9 (struct expent *lval)
+static int hie9 (ExprDesc *lval)
 /* Process * and / operators. */
 {
-    static GenDesc* hie9_ops [] = {
+    static const GenDesc* hie9_ops [] = {
        &GenMUL, &GenDIV, &GenMOD, 0
     };
     int UsedGen;
@@ -1882,13 +1953,13 @@ static int hie9 (struct expent *lval)
 
 
 
-static void parseadd (int k, struct expent* lval)
+static void parseadd (int k, ExprDesc* lval)
 /* Parse an expression with the binary plus operator. lval contains the
  * unprocessed left hand side of the expression and will contain the
  * result of the expression on return.
  */
 {
-    struct expent lval2;
+    ExprDesc lval2;
     unsigned flags;            /* Operation flags */
     CodeMark Mark;             /* Remember code position */
     type* lhst;                        /* Type of left hand side */
@@ -1899,31 +1970,32 @@ static void parseadd (int k, struct expent* lval)
     NextToken ();
 
     /* Get the left hand side type, initialize operation flags */
-    lhst = lval->e_tptr;
+    lhst = lval->Type;
     flags = 0;
 
     /* Check for constness on both sides */
-    if (k == 0 && lval->e_flags == E_MCONST) {
+    if (k == 0 && (lval->Flags & E_MCONST) != 0) {
 
        /* The left hand side is a constant. Good. Get rhs */
-               if (evalexpr (CF_NONE, hie9, &lval2) == 0) {
+       k = hie9 (&lval2);
+               if (k == 0 && lval2.Flags == E_MCONST) {
 
                    /* Right hand side is also constant. Get the rhs type */
-           rhst = lval2.e_tptr;
+           rhst = lval2.Type;
 
            /* Both expressions are constants. Check for pointer arithmetic */
                    if (IsClassPtr (lhst) && IsClassInt (rhst)) {
                        /* Left is pointer, right is int, must scale rhs */
-               lval->e_const = lval->e_const + lval2.e_const * PSizeOf (lhst);
+                       lval->ConstVal += lval2.ConstVal * CheckedPSizeOf (lhst);
                /* Result type is a pointer */
            } else if (IsClassInt (lhst) && IsClassPtr (rhst)) {
                /* Left is int, right is pointer, must scale lhs */
-                       lval->e_const = lval->e_const * PSizeOf (rhst) + lval2.e_const;
+                       lval->ConstVal = lval->ConstVal * CheckedPSizeOf (rhst) + lval2.ConstVal;
                /* Result type is a pointer */
-               lval->e_tptr = lval2.e_tptr;
+               lval->Type = lval2.Type;
                    } else if (IsClassInt (lhst) && IsClassInt (rhst)) {
                /* Integer addition */
-               lval->e_const += lval2.e_const;
+               lval->ConstVal += lval2.ConstVal;
                typeadjust (lval, &lval2, 1);
            } else {
                        /* OOPS */
@@ -1931,39 +2003,93 @@ static void parseadd (int k, struct expent* lval)
            }
 
                    /* Result is constant, condition codes not set */
-                   lval->e_test = E_MCONST;
+                   lval->Test &= ~E_CC;
 
        } else {
 
-           /* lhs is constant, rhs is not. Get the rhs type. */
-           rhst = lval2.e_tptr;
+           /* lhs is a constant and rhs is not constant. Load rhs into
+            * the primary.
+            */
+           exprhs (CF_NONE, k, &lval2);
+
+                   /* Beware: The check above (for lhs) lets not only pass numeric
+            * constants, but also constant addresses (labels), maybe even
+            * with an offset. We have to check for that here.
+            */
+
+           /* First, get the rhs type. */
+           rhst = lval2.Type;
+
+           /* Setup flags */
+           if (lval->Flags == E_MCONST) {
+               /* A numerical constant */
+               flags |= CF_CONST;
+           } else {
+               /* Constant address label */
+               flags |= GlobalModeFlags (lval->Flags) | CF_CONSTADDR;
+           }
 
            /* Check for pointer arithmetic */
            if (IsClassPtr (lhst) && IsClassInt (rhst)) {
                /* Left is pointer, right is int, must scale rhs */
-               g_scale (CF_INT, PSizeOf (lhst));
+               g_scale (CF_INT, CheckedPSizeOf (lhst));
                /* Operate on pointers, result type is a pointer */
-               flags = CF_PTR;
+               flags |= CF_PTR;
+               /* Generate the code for the add */
+               if (lval->Flags == E_MCONST) {
+                   /* Numeric constant */
+                   g_inc (flags, lval->ConstVal);
+               } else {
+                   /* Constant address */
+                   g_addaddr_static (flags, lval->Name, lval->ConstVal);
+               }
            } else if (IsClassInt (lhst) && IsClassPtr (rhst)) {
-               /* Left is int, right is pointer, must scale lhs */
-                       lval->e_const *= PSizeOf (rhst);
-               /* Operate on pointers, result type is a pointer */
-               flags = CF_PTR;
-               lval->e_tptr = lval2.e_tptr;
+
+               /* Left is int, right is pointer, must scale lhs. */
+               unsigned ScaleFactor = CheckedPSizeOf (rhst);
+
+                       /* Operate on pointers, result type is a pointer */
+               flags |= CF_PTR;
+               lval->Type = lval2.Type;
+
+               /* Since we do already have rhs in the primary, if lhs is
+                * not a numeric constant, and the scale factor is not one
+                * (no scaling), we must take the long way over the stack.
+                */
+               if (lval->Flags == E_MCONST) {
+                   /* Numeric constant, scale lhs */
+                   lval->ConstVal *= ScaleFactor;
+                   /* Generate the code for the add */
+                   g_inc (flags, lval->ConstVal);
+               } else if (ScaleFactor == 1) {
+                   /* Constant address but no need to scale */
+                   g_addaddr_static (flags, lval->Name, lval->ConstVal);
+               } else {
+                   /* Constant address that must be scaled */
+                           g_push (TypeOf (lval2.Type), 0);    /* rhs --> stack */
+                   g_getimmed (flags, lval->Name, lval->ConstVal);
+                   g_scale (CF_PTR, ScaleFactor);
+                   g_add (CF_PTR, 0);
+               }
                    } else if (IsClassInt (lhst) && IsClassInt (rhst)) {
                /* Integer addition */
-                       flags = typeadjust (lval, &lval2, 1);
+                       flags |= typeadjust (lval, &lval2, 1);
+               /* Generate the code for the add */
+               if (lval->Flags == E_MCONST) {
+                   /* Numeric constant */
+                   g_inc (flags, lval->ConstVal);
+               } else {
+                   /* Constant address */
+                   g_addaddr_static (flags, lval->Name, lval->ConstVal);
+               }
            } else {
                        /* OOPS */
                Error ("Invalid operands for binary operator `+'");
                    }
 
-           /* Generate code for the add */
-           g_inc (flags | CF_CONST, lval->e_const);
-
            /* Result is in primary register */
-           lval->e_flags = E_MEXPR;
-           lval->e_test &= ~E_CC;
+           lval->Flags = E_MEXPR;
+           lval->Test &= ~E_CC;
 
                }
 
@@ -1972,30 +2098,30 @@ static void parseadd (int k, struct expent* lval)
        /* Left hand side is not constant. Get the value onto the stack. */
        exprhs (CF_NONE, k, lval);              /* --> primary register */
                Mark = GetCodePos ();
-       g_push (TypeOf (lval->e_tptr), 0);      /* --> stack */
+       g_push (TypeOf (lval->Type), 0);        /* --> stack */
 
        /* Evaluate the rhs */
                if (evalexpr (CF_NONE, hie9, &lval2) == 0) {
 
                    /* Right hand side is a constant. Get the rhs type */
-           rhst = lval2.e_tptr;
+           rhst = lval2.Type;
 
            /* Remove pushed value from stack */
            RemoveCode (Mark);
-           pop (TypeOf (lval->e_tptr));
+           pop (TypeOf (lval->Type));
 
                    /* Check for pointer arithmetic */
                    if (IsClassPtr (lhst) && IsClassInt (rhst)) {
                /* Left is pointer, right is int, must scale rhs */
-               lval2.e_const *= PSizeOf (lhst);
+               lval2.ConstVal *= CheckedPSizeOf (lhst);
                /* Operate on pointers, result type is a pointer */
                flags = CF_PTR;
            } else if (IsClassInt (lhst) && IsClassPtr (rhst)) {
                /* Left is int, right is pointer, must scale lhs (ptr only) */
-                       g_scale (CF_INT | CF_CONST, PSizeOf (rhst));
+                       g_scale (CF_INT | CF_CONST, CheckedPSizeOf (rhst));
                        /* Operate on pointers, result type is a pointer */
                flags = CF_PTR;
-               lval->e_tptr = lval2.e_tptr;
+               lval->Type = lval2.Type;
                    } else if (IsClassInt (lhst) && IsClassInt (rhst)) {
                /* Integer addition */
                flags = typeadjust (lval, &lval2, 1);
@@ -2005,34 +2131,42 @@ static void parseadd (int k, struct expent* lval)
            }
 
            /* Generate code for the add */
-                   g_inc (flags | CF_CONST, lval2.e_const);
+                   g_inc (flags | CF_CONST, lval2.ConstVal);
 
            /* Result is in primary register */
-           lval->e_flags = E_MEXPR;
-           lval->e_test &= ~E_CC;
+           lval->Flags = E_MEXPR;
+           lval->Test &= ~E_CC;
 
        } else {
 
            /* lhs and rhs are not constant. Get the rhs type. */
-           rhst = lval2.e_tptr;
+           rhst = lval2.Type;
 
            /* Check for pointer arithmetic */
            if (IsClassPtr (lhst) && IsClassInt (rhst)) {
                /* Left is pointer, right is int, must scale rhs */
-               g_scale (CF_INT, PSizeOf (lhst));
+               g_scale (CF_INT, CheckedPSizeOf (lhst));
                /* Operate on pointers, result type is a pointer */
                flags = CF_PTR;
            } else if (IsClassInt (lhst) && IsClassPtr (rhst)) {
                /* Left is int, right is pointer, must scale lhs */
                g_tosint (TypeOf (rhst));       /* Make sure, TOS is int */
                g_swap (CF_INT);                /* Swap TOS and primary */
-               g_scale (CF_INT, PSizeOf (rhst));
+               g_scale (CF_INT, CheckedPSizeOf (rhst));
                /* Operate on pointers, result type is a pointer */
                flags = CF_PTR;
-               lval->e_tptr = lval2.e_tptr;
+               lval->Type = lval2.Type;
                    } else if (IsClassInt (lhst) && IsClassInt (rhst)) {
-               /* Integer addition */
-                       flags = typeadjust (lval, &lval2, 0);
+               /* Integer addition. Note: Result is never constant.
+                 * Problem here is that typeadjust does not know if the
+                 * variable is an rvalue or lvalue, so if both operands
+                 * are dereferenced constant numeric addresses, typeadjust
+                 * thinks the operation works on constants. Removing
+                 * CF_CONST here means handling the symptoms, however, the
+                 * whole parser is such a mess that I fear to break anything
+                 * when trying to apply another solution.
+                 */
+                       flags = typeadjust (lval, &lval2, 0) & ~CF_CONST;
            } else {
                        /* OOPS */
                Error ("Invalid operands for binary operator `+'");
@@ -2042,8 +2176,8 @@ static void parseadd (int k, struct expent* lval)
                    g_add (flags, 0);
 
            /* Result is in primary register */
-                   lval->e_flags = E_MEXPR;
-           lval->e_test &= ~E_CC;
+                   lval->Flags = E_MEXPR;
+           lval->Test &= ~E_CC;
 
                }
 
@@ -2052,13 +2186,13 @@ static void parseadd (int k, struct expent* lval)
 
 
 
-static void parsesub (int k, struct expent* lval)
+static void parsesub (int k, ExprDesc* lval)
 /* Parse an expression with the binary minus operator. lval contains the
  * unprocessed left hand side of the expression and will contain the
  * result of the expression on return.
  */
 {
-    struct expent lval2;
+    ExprDesc lval2;
     unsigned flags;            /* Operation flags */
     type* lhst;                        /* Type of left hand side */
     type* rhst;                        /* Type of right hand side */
@@ -2071,7 +2205,7 @@ static void parsesub (int k, struct expent* lval)
     NextToken ();
 
     /* Get the left hand side type, initialize operation flags */
-    lhst = lval->e_tptr;
+    lhst = lval->Type;
     flags = 0;
     rscale = 1;                        /* Scale by 1, that is, don't scale */
 
@@ -2085,10 +2219,10 @@ static void parsesub (int k, struct expent* lval)
     if (evalexpr (CF_NONE, hie9, &lval2) == 0) {
 
        /* The right hand side is constant. Get the rhs type. */
-               rhst = lval2.e_tptr;
+               rhst = lval2.Type;
 
        /* Check left hand side */
-       if (k == 0 && lval->e_flags & E_MCONST) {
+       if (k == 0 && (lval->Flags & E_MCONST) != 0) {
 
            /* Both sides are constant, remove generated code */
            RemoveCode (Mark1);
@@ -2097,29 +2231,30 @@ static void parsesub (int k, struct expent* lval)
            /* Check for pointer arithmetic */
            if (IsClassPtr (lhst) && IsClassInt (rhst)) {
                /* Left is pointer, right is int, must scale rhs */
-               lval->e_const -= lval2.e_const * PSizeOf (lhst);
+               lval->ConstVal -= lval2.ConstVal * CheckedPSizeOf (lhst);
                /* Operate on pointers, result type is a pointer */
            } else if (IsClassPtr (lhst) && IsClassPtr (rhst)) {
                /* Left is pointer, right is pointer, must scale result */
-               if (TypeCmp (Indirect (lhst), Indirect (rhst)) < TC_EQUAL) {
+               if (TypeCmp (Indirect (lhst), Indirect (rhst)) < TC_QUAL_DIFF) {
                    Error ("Incompatible pointer types");
                } else {
-                   lval->e_const = (lval->e_const - lval2.e_const) / PSizeOf (lhst);
+                   lval->ConstVal = (lval->ConstVal - lval2.ConstVal) /
+                                      CheckedPSizeOf (lhst);
                }
                /* Operate on pointers, result type is an integer */
-               lval->e_tptr = type_int;
+               lval->Type = type_int;
            } else if (IsClassInt (lhst) && IsClassInt (rhst)) {
                /* Integer subtraction */
                        typeadjust (lval, &lval2, 1);
-               lval->e_const -= lval2.e_const;
+               lval->ConstVal -= lval2.ConstVal;
            } else {
                /* OOPS */
                Error ("Invalid operands for binary operator `-'");
            }
 
            /* Result is constant, condition codes not set */
-           lval->e_flags = E_MCONST;
-           lval->e_test &= ~E_CC;
+           /* lval->Flags = E_MCONST; ### */
+           lval->Test &= ~E_CC;
 
        } else {
 
@@ -2131,19 +2266,19 @@ static void parsesub (int k, struct expent* lval)
 
            if (IsClassPtr (lhst) && IsClassInt (rhst)) {
                /* Left is pointer, right is int, must scale rhs */
-                       lval2.e_const *= PSizeOf (lhst);
+                       lval2.ConstVal *= CheckedPSizeOf (lhst);
                /* Operate on pointers, result type is a pointer */
                flags = CF_PTR;
            } else if (IsClassPtr (lhst) && IsClassPtr (rhst)) {
                /* Left is pointer, right is pointer, must scale result */
-               if (TypeCmp (Indirect (lhst), Indirect (rhst)) < TC_EQUAL) {
+               if (TypeCmp (Indirect (lhst), Indirect (rhst)) < TC_QUAL_DIFF) {
                    Error ("Incompatible pointer types");
                } else {
-                   rscale = PSizeOf (lhst);
+                   rscale = CheckedPSizeOf (lhst);
                }
                /* Operate on pointers, result type is an integer */
                flags = CF_PTR;
-               lval->e_tptr = type_int;
+               lval->Type = type_int;
            } else if (IsClassInt (lhst) && IsClassInt (rhst)) {
                /* Integer subtraction */
                        flags = typeadjust (lval, &lval2, 1);
@@ -2153,7 +2288,7 @@ static void parsesub (int k, struct expent* lval)
            }
 
            /* Do the subtraction */
-           g_dec (flags | CF_CONST, lval2.e_const);
+           g_dec (flags | CF_CONST, lval2.ConstVal);
 
            /* If this was a pointer subtraction, we must scale the result */
            if (rscale != 1) {
@@ -2161,39 +2296,39 @@ static void parsesub (int k, struct expent* lval)
            }
 
            /* Result is in primary register */
-           lval->e_flags = E_MEXPR;
-           lval->e_test &= ~E_CC;
+           lval->Flags = E_MEXPR;
+           lval->Test &= ~E_CC;
 
        }
 
     } else {
 
        /* Right hand side is not constant. Get the rhs type. */
-       rhst = lval2.e_tptr;
+       rhst = lval2.Type;
 
                /* Check for pointer arithmetic */
        if (IsClassPtr (lhst) && IsClassInt (rhst)) {
            /* Left is pointer, right is int, must scale rhs */
-           g_scale (CF_INT, PSizeOf (lhst));
+           g_scale (CF_INT, CheckedPSizeOf (lhst));
            /* Operate on pointers, result type is a pointer */
            flags = CF_PTR;
        } else if (IsClassPtr (lhst) && IsClassPtr (rhst)) {
            /* Left is pointer, right is pointer, must scale result */
-           if (TypeCmp (Indirect (lhst), Indirect (rhst)) < TC_EQUAL) {
+           if (TypeCmp (Indirect (lhst), Indirect (rhst)) < TC_QUAL_DIFF) {
                Error ("Incompatible pointer types");
            } else {
-               rscale = PSizeOf (lhst);
+               rscale = CheckedPSizeOf (lhst);
            }
            /* Operate on pointers, result type is an integer */
            flags = CF_PTR;
-           lval->e_tptr = type_int;
+           lval->Type = type_int;
        } else if (IsClassInt (lhst) && IsClassInt (rhst)) {
            /* Integer subtraction. If the left hand side descriptor says that
             * the lhs is const, we have to remove this mark, since this is no
             * longer true, lhs is on stack instead.
             */
-           if (lval->e_flags == E_MCONST) {
-               lval->e_flags = E_MEXPR;
+           if (lval->Flags == E_MCONST) {
+               lval->Flags = E_MEXPR;
            }
            /* Adjust operand types */
            flags = typeadjust (lval, &lval2, 0);
@@ -2211,20 +2346,20 @@ static void parsesub (int k, struct expent* lval)
        }
 
        /* Result is in primary register */
-       lval->e_flags = E_MEXPR;
-       lval->e_test &= ~E_CC;
+       lval->Flags = E_MEXPR;
+       lval->Test &= ~E_CC;
     }
 }
 
 
 
-static int hie8 (struct expent* lval)
+static int hie8 (ExprDesc* lval)
 /* Process + and - binary operators. */
 {
     int k = hie9 (lval);
-    while (curtok == TOK_PLUS || curtok == TOK_MINUS) {
+    while (CurTok.Tok == TOK_PLUS || CurTok.Tok == TOK_MINUS) {
 
-               if (curtok == TOK_PLUS) {
+               if (CurTok.Tok == TOK_PLUS) {
                    parseadd (k, lval);
                } else {
                    parsesub (k, lval);
@@ -2237,10 +2372,10 @@ static int hie8 (struct expent* lval)
 
 
 
-static int hie7 (struct expent *lval)
+static int hie7 (ExprDesc *lval)
 /* Parse << and >>. */
 {
-    static GenDesc* hie7_ops [] = {
+    static const GenDesc* hie7_ops [] = {
                &GenASL, &GenASR, 0
     };
     int UsedGen;
@@ -2250,10 +2385,10 @@ static int hie7 (struct expent *lval)
 
 
 
-static int hie6 (struct expent *lval)
+static int hie6 (ExprDesc *lval)
 /* process greater-than type comparators */
 {
-    static GenDesc* hie6_ops [] = {
+    static const GenDesc* hie6_ops [] = {
        &GenLT, &GenLE, &GenGE, &GenGT, 0
     };
     return hie_compare (hie6_ops, lval, hie7);
@@ -2261,9 +2396,9 @@ static int hie6 (struct expent *lval)
 
 
 
-static int hie5 (struct expent *lval)
+static int hie5 (ExprDesc *lval)
 {
-    static GenDesc* hie5_ops[] = {
+    static const GenDesc* hie5_ops[] = {
                &GenEQ, &GenNE, 0
     };
     return hie_compare (hie5_ops, lval, hie6);
@@ -2271,10 +2406,10 @@ static int hie5 (struct expent *lval)
 
 
 
-static int hie4 (struct expent* lval)
+static int hie4 (ExprDesc* lval)
 /* Handle & (bitwise and) */
 {
-    static GenDesc* hie4_ops [] = {
+    static const GenDesc* hie4_ops [] = {
                &GenAND, 0
     };
     int UsedGen;
@@ -2284,10 +2419,10 @@ static int hie4 (struct expent* lval)
 
 
 
-static int hie3 (struct expent *lval)
+static int hie3 (ExprDesc *lval)
 /* Handle ^ (bitwise exclusive or) */
 {
-    static GenDesc* hie3_ops [] = {
+    static const GenDesc* hie3_ops [] = {
                &GenXOR, 0
     };
     int UsedGen;
@@ -2297,10 +2432,10 @@ static int hie3 (struct expent *lval)
 
 
 
-static int hie2 (struct expent *lval)
+static int hie2 (ExprDesc *lval)
 /* Handle | (bitwise or) */
 {
-    static GenDesc* hie2_ops [] = {
+    static const GenDesc* hie2_ops [] = {
                &GenOR, 0
     };
     int UsedGen;
@@ -2310,25 +2445,101 @@ static int hie2 (struct expent *lval)
 
 
 
-static int hieAnd (struct expent* lval, unsigned TrueLab, int* BoolOp)
+static int hieAndPP (ExprDesc* lval)
+/* Process "exp && exp" in preprocessor mode (that is, when the parser is
+ * called recursively from the preprocessor.
+ */
+{
+    ExprDesc lval2;
+
+    ConstSubExpr (hie2, lval);
+    while (CurTok.Tok == TOK_BOOL_AND) {
+
+       /* Left hand side must be an int */
+       if (!IsClassInt (lval->Type)) {
+           Error ("Left hand side must be of integer type");
+           MakeConstIntExpr (lval, 1);
+       }
+
+       /* Skip the && */
+       NextToken ();
+
+       /* Get rhs */
+       ConstSubExpr (hie2, &lval2);
+
+       /* Since we are in PP mode, all we know about is integers */
+       if (!IsClassInt (lval2.Type)) {
+           Error ("Right hand side must be of integer type");
+           MakeConstIntExpr (&lval2, 1);
+       }
+
+       /* Combine the two */
+       lval->ConstVal = (lval->ConstVal && lval2.ConstVal);
+    }
+
+    /* Always a rvalue */
+    return 0;
+}
+
+
+
+static int hieOrPP (ExprDesc *lval)
+/* Process "exp || exp" in preprocessor mode (that is, when the parser is
+ * called recursively from the preprocessor.
+ */
+{
+    ExprDesc lval2;
+
+    ConstSubExpr (hieAndPP, lval);
+    while (CurTok.Tok == TOK_BOOL_OR) {
+
+       /* Left hand side must be an int */
+       if (!IsClassInt (lval->Type)) {
+           Error ("Left hand side must be of integer type");
+           MakeConstIntExpr (lval, 1);
+       }
+
+       /* Skip the && */
+       NextToken ();
+
+       /* Get rhs */
+       ConstSubExpr (hieAndPP, &lval2);
+
+       /* Since we are in PP mode, all we know about is integers */
+       if (!IsClassInt (lval2.Type)) {
+           Error ("Right hand side must be of integer type");
+           MakeConstIntExpr (&lval2, 1);
+       }
+
+       /* Combine the two */
+       lval->ConstVal = (lval->ConstVal || lval2.ConstVal);
+    }
+
+    /* Always a rvalue */
+    return 0;
+}
+
+
+
+static int hieAnd (ExprDesc* lval, unsigned TrueLab, int* BoolOp)
 /* Process "exp && exp" */
 {
     int k;
     int lab;
-    struct expent lval2;
+    ExprDesc lval2;
 
     k = hie2 (lval);
-    if (curtok == TOK_BOOL_AND) {
+    if (CurTok.Tok == TOK_BOOL_AND) {
 
                /* Tell our caller that we're evaluating a boolean */
                *BoolOp = 1;
 
                /* Get a label that we will use for false expressions */
-               lab = GetLabel ();
+               lab = GetLocalLabel ();
 
                /* If the expr hasn't set condition codes, set the force-test flag */
-               if ((lval->e_test & E_CC) == 0) {
-                   lval->e_test |= E_FORCETEST;
+               if ((lval->Test & E_CC) == 0) {
+                   lval->Test |= E_FORCETEST;
                }
 
                /* Load the value */
@@ -2338,20 +2549,20 @@ static int hieAnd (struct expent* lval, unsigned TrueLab, int* BoolOp)
                g_falsejump (CF_NONE, lab);
 
                /* Parse more boolean and's */
-               while (curtok == TOK_BOOL_AND) {
+               while (CurTok.Tok == TOK_BOOL_AND) {
 
                    /* Skip the && */
            NextToken ();
 
            /* Get rhs */
            k = hie2 (&lval2);
-           if ((lval2.e_test & E_CC) == 0) {
-               lval2.e_test |= E_FORCETEST;
+           if ((lval2.Test & E_CC) == 0) {
+               lval2.Test |= E_FORCETEST;
            }
            exprhs (CF_FORCECHAR, k, &lval2);
 
                    /* Do short circuit evaluation */
-           if (curtok == TOK_BOOL_AND) {
+           if (CurTok.Tok == TOK_BOOL_AND) {
                g_falsejump (CF_NONE, lab);
                    } else {
                        /* Last expression - will evaluate to true */
@@ -2360,11 +2571,11 @@ static int hieAnd (struct expent* lval, unsigned TrueLab, int* BoolOp)
                }
 
                /* Define the false jump label here */
-               g_defloclabel (lab);
+               g_defcodelabel (lab);
 
                /* Define the label */
-               lval->e_flags = E_MEXPR;
-               lval->e_test |= E_CC;   /* Condition codes are set */
+               lval->Flags = E_MEXPR;
+               lval->Test |= E_CC;     /* Condition codes are set */
                k = 0;
     }
     return k;
@@ -2372,28 +2583,28 @@ static int hieAnd (struct expent* lval, unsigned TrueLab, int* BoolOp)
 
 
 
-static int hieOr (struct expent *lval)
+static int hieOr (ExprDesc *lval)
 /* Process "exp || exp". */
 {
     int k;
-    struct expent lval2;
+    ExprDesc lval2;
     int BoolOp = 0;            /* Did we have a boolean op? */
     int AndOp;                 /* Did we have a && operation? */
     unsigned TrueLab;          /* Jump to this label if true */
     unsigned DoneLab;
 
     /* Get a label */
-    TrueLab = GetLabel ();
+    TrueLab = GetLocalLabel ();
 
     /* Call the next level parser */
     k = hieAnd (lval, TrueLab, &BoolOp);
 
     /* Any boolean or's? */
-    if (curtok == TOK_BOOL_OR) {
+    if (CurTok.Tok == TOK_BOOL_OR) {
 
        /* If the expr hasn't set condition codes, set the force-test flag */
-               if ((lval->e_test & E_CC) == 0) {
-           lval->e_test |= E_FORCETEST;
+               if ((lval->Test & E_CC) == 0) {
+           lval->Test |= E_FORCETEST;
        }
 
        /* Get first expr */
@@ -2410,7 +2621,7 @@ static int hieOr (struct expent *lval)
        BoolOp = 1;
 
        /* while there's more expr */
-               while (curtok == TOK_BOOL_OR) {
+               while (CurTok.Tok == TOK_BOOL_OR) {
 
                    /* skip the || */
            NextToken ();
@@ -2418,77 +2629,77 @@ static int hieOr (struct expent *lval)
                    /* Get a subexpr */
            AndOp = 0;
            k = hieAnd (&lval2, TrueLab, &AndOp);
-                   if ((lval2.e_test & E_CC) == 0) {
-               lval2.e_test |= E_FORCETEST;
+                   if ((lval2.Test & E_CC) == 0) {
+               lval2.Test |= E_FORCETEST;
            }
            exprhs (CF_FORCECHAR, k, &lval2);
 
-                   /* If there is more to come, add shortcut boolean eval.
-            * Beware: If we had && operators, the jump is already
-            * in place!
-            */
-#if    0
-/* Seems this sometimes generates wrong code */
-           if (curtok == TOK_BOOL_OR && !AndOp) {
-               g_truejump (CF_NONE, TrueLab);
-                   }
-#else
+                   /* If there is more to come, add shortcut boolean eval. */
            g_truejump (CF_NONE, TrueLab);
-#endif
-       }
-       lval->e_flags = E_MEXPR;
-       lval->e_test |= E_CC;                   /* Condition codes are set */
+
+       }
+       lval->Flags = E_MEXPR;
+       lval->Test |= E_CC;                     /* Condition codes are set */
        k = 0;
     }
 
     /* If we really had boolean ops, generate the end sequence */
     if (BoolOp) {
-       DoneLab = GetLabel ();
+       DoneLab = GetLocalLabel ();
        g_getimmed (CF_INT | CF_CONST, 0, 0);   /* Load FALSE */
                g_falsejump (CF_NONE, DoneLab);
-       g_defloclabel (TrueLab);
+       g_defcodelabel (TrueLab);
        g_getimmed (CF_INT | CF_CONST, 1, 0);   /* Load TRUE */
-       g_defloclabel (DoneLab);
+       g_defcodelabel (DoneLab);
     }
     return k;
 }
 
 
 
-static int hieQuest (struct expent *lval)
+static int hieQuest (ExprDesc *lval)
 /* Parse "lvalue ? exp : exp" */
 {
     int k;
     int labf;
     int labt;
-    struct expent lval2;               /* Expression 2 */
-    struct expent lval3;               /* Expression 3 */
+    ExprDesc lval2;            /* Expression 2 */
+    ExprDesc lval3;            /* Expression 3 */
     type* type2;               /* Type of expression 2 */
     type* type3;               /* Type of expression 3 */
     type* rtype;               /* Type of result */
-    CodeMark Mark1;            /* Save position in output code */
-    CodeMark Mark2;            /* Save position in output code */
 
 
-
-    k = hieOr (lval);
-    if (curtok == TOK_QUEST) {
+    k = Preprocessing? hieOrPP (lval) : hieOr (lval);
+    if (CurTok.Tok == TOK_QUEST) {
        NextToken ();
-       if ((lval->e_test & E_CC) == 0) {
+       if ((lval->Test & E_CC) == 0) {
            /* Condition codes not set, force a test */
-           lval->e_test |= E_FORCETEST;
+           lval->Test |= E_FORCETEST;
        }
        exprhs (CF_NONE, k, lval);
-       labf = GetLabel ();
+       labf = GetLocalLabel ();
        g_falsejump (CF_NONE, labf);
 
-       /* Parse second and third expression */
-       expression1 (&lval2);
-       labt = GetLabel ();
+       /* Parse second expression */
+        k = expr (hie1, &lval2);
+       type2 = lval2.Type;
+        if (!IsTypeVoid (lval2.Type)) {
+            /* Load it into the primary */
+            exprhs (CF_NONE, k, &lval2);
+        }
+       labt = GetLocalLabel ();
        ConsumeColon ();
        g_jump (labt);
-       g_defloclabel (labf);
-       expression1 (&lval3);
+
+        /* Parse the third expression */
+       g_defcodelabel (labf);
+        k = expr (hie1, &lval3);
+       type3 = lval3.Type;
+        if (!IsTypeVoid (lval3.Type)) {
+            /* Load it into the primary */
+            exprhs (CF_NONE, k, &lval3);
+        }
 
        /* Check if any conversions are needed, if so, do them.
         * Conversion rules for ?: expression are:
@@ -2499,10 +2710,10 @@ static int hieQuest (struct expent *lval)
         *   - if one of the expressions is a pointer and the other is
         *     a zero constant, the resulting type is that of the pointer
         *     type.
+         *   - if both expressions are void expressions, the result is of
+         *     type void.
         *   - all other cases are flagged by an error.
         */
-       type2 = lval2.e_tptr;
-       type3 = lval3.e_tptr;
        if (IsClassInt (type2) && IsClassInt (type3)) {
 
            /* Get common type */
@@ -2514,54 +2725,48 @@ static int hieQuest (struct expent *lval)
            /* Setup a new label so that the expr3 code will jump around
             * the type cast code for expr2.
             */
-                   labf = GetLabel ();         /* Get new label */
-           Mark1 = GetCodePos ();      /* Remember current position */
+                   labf = GetLocalLabel ();    /* Get new label */
            g_jump (labf);              /* Jump around code */
 
            /* The jump for expr2 goes here */
-           g_defloclabel (labt);
+           g_defcodelabel (labt);
 
            /* Create the typecast code for expr2 */
-           Mark2 = GetCodePos ();      /* Remember position */
            g_typecast (TypeOf (rtype), TypeOf (type2));
 
-           /* If the typecast did not produce code, remove the jump,
-            * otherwise output the label.
-            */
-           if (GetCodePos() == Mark2) {
-               RemoveCode (Mark1);     /* Remove code */
-           } else {
-               /* We have typecast code, output label */
-               g_defloclabel (labf);
-               labt = 0;               /* Mark other label as invalid */
-           }
+           /* Jump here around the typecase code. */
+           g_defcodelabel (labf);
+           labt = 0;           /* Mark other label as invalid */
 
        } else if (IsClassPtr (type2) && IsClassPtr (type3)) {
            /* Must point to same type */
            if (TypeCmp (Indirect (type2), Indirect (type3)) < TC_EQUAL) {
-               Error ("Incompatible pointer types");
+               Error ("Incompatible pointer types");
            }
            /* Result has the common type */
-           rtype = lval2.e_tptr;
+           rtype = lval2.Type;
        } else if (IsClassPtr (type2) && IsNullPtr (&lval3)) {
            /* Result type is pointer, no cast needed */
-           rtype = lval2.e_tptr;
+           rtype = lval2.Type;
        } else if (IsNullPtr (&lval2) && IsClassPtr (type3)) {
            /* Result type is pointer, no cast needed */
-           rtype = lval3.e_tptr;
+           rtype = lval3.Type;
+        } else if (IsTypeVoid (type2) && IsTypeVoid (type3)) {
+            /* Result type is void */
+            rtype = lval3.Type;
        } else {
            Error ("Incompatible types");
-           rtype = lval2.e_tptr;               /* Doesn't matter here */
+           rtype = lval2.Type;         /* Doesn't matter here */
        }
 
        /* If we don't have the label defined until now, do it */
        if (labt) {
-           g_defloclabel (labt);
+           g_defcodelabel (labt);
        }
 
        /* Setup the target expression */
-               lval->e_flags = E_MEXPR;
-       lval->e_tptr = rtype;
+               lval->Flags = E_MEXPR;
+       lval->Type = rtype;
        k = 0;
     }
     return k;
@@ -2569,10 +2774,10 @@ static int hieQuest (struct expent *lval)
 
 
 
-static void opeq (GenDesc* Gen, struct expent *lval, int k)
+static void opeq (const GenDesc* Gen, ExprDesc *lval, int k)
 /* Process "op=" operators. */
 {
-    struct expent lval2;
+    ExprDesc lval2;
     unsigned flags;
     CodeMark Mark;
     int MustScale;
@@ -2584,9 +2789,9 @@ static void opeq (GenDesc* Gen, struct expent *lval, int k)
     }
 
     /* Determine the type of the lhs */
-    flags = TypeOf (lval->e_tptr);
+    flags = TypeOf (lval->Type);
     MustScale = (Gen->Func == g_add || Gen->Func == g_sub) &&
-               lval->e_tptr [0] == T_PTR;
+               lval->Type [0] == T_PTR;
 
     /* Get the lhs address on stack (if needed) */
     PushAddr (lval);
@@ -2609,65 +2814,66 @@ static void opeq (GenDesc* Gen, struct expent *lval, int k)
        }
                if (MustScale) {
            /* lhs is a pointer, scale rhs */
-           lval2.e_const *= SizeOf (lval->e_tptr+1);
+           lval2.ConstVal *= CheckedSizeOf (lval->Type+1);
        }
 
        /* If the lhs is character sized, the operation may be later done
         * with characters.
         */
-       if (SizeOf (lval->e_tptr) == 1) {
+       if (CheckedSizeOf (lval->Type) == SIZEOF_CHAR) {
            flags |= CF_FORCECHAR;
        }
 
        /* Special handling for add and sub - some sort of a hack, but short code */
        if (Gen->Func == g_add) {
-           g_inc (flags | CF_CONST, lval2.e_const);
+           g_inc (flags | CF_CONST, lval2.ConstVal);
        } else if (Gen->Func == g_sub) {
-           g_dec (flags | CF_CONST, lval2.e_const);
+           g_dec (flags | CF_CONST, lval2.ConstVal);
        } else {
-                   Gen->Func (flags | CF_CONST, lval2.e_const);
+                   Gen->Func (flags | CF_CONST, lval2.ConstVal);
        }
     } else {
        /* rhs is not constant and already in the primary register */
                if (MustScale) {
            /* lhs is a pointer, scale rhs */
-                   g_scale (TypeOf (lval2.e_tptr), SizeOf (lval->e_tptr+1));
+                   g_scale (TypeOf (lval2.Type), CheckedSizeOf (lval->Type+1));
        }
 
        /* If the lhs is character sized, the operation may be later done
         * with characters.
         */
-       if (SizeOf (lval->e_tptr) == 1) {
+       if (CheckedSizeOf (lval->Type) == SIZEOF_CHAR) {
            flags |= CF_FORCECHAR;
        }
 
        /* Adjust the types of the operands if needed */
-               Gen->Func (g_typeadjust (flags, TypeOf (lval2.e_tptr)), 0);
+               Gen->Func (g_typeadjust (flags, TypeOf (lval2.Type)), 0);
     }
-    store (lval);
-    lval->e_flags = E_MEXPR;
+    Store (lval, 0);
+    lval->Flags = E_MEXPR;
 }
 
 
 
-static void addsubeq (GenDesc* Gen, struct expent *lval, int k)
+static void addsubeq (const GenDesc* Gen, ExprDesc *lval, int k)
 /* Process the += and -= operators */
 {
-    struct expent lval2;
-    unsigned flags;
+    ExprDesc lval2;
+    unsigned lflags;
+    unsigned rflags;
     int MustScale;
 
 
+    /* We must have an lvalue */
     if (k == 0) {
        Error ("Invalid lvalue in assignment");
        return;
     }
 
-
     /* We're currently only able to handle some adressing modes */
-    if ((lval->e_flags & E_MGLOBAL) == 0 &&    /* Global address? */
-       (lval->e_flags & E_MLOCAL) == 0  &&     /* Local address? */
-               (lval->e_flags & E_MCONST) == 0) {      /* Constant address? */
+    if ((lval->Flags & E_MGLOBAL) == 0 &&      /* Global address? */
+       (lval->Flags & E_MLOCAL) == 0  &&       /* Local address? */
+               (lval->Flags & E_MCONST) == 0) {        /* Constant address? */
        /* Use generic routine */
                opeq (Gen, lval, k);
        return;
@@ -2677,155 +2883,84 @@ static void addsubeq (GenDesc* Gen, struct expent *lval, int k)
     NextToken ();
 
     /* Check if we have a pointer expression and must scale rhs */
-    MustScale = (lval->e_tptr [0] == T_PTR);
+    MustScale = (lval->Type [0] == T_PTR);
 
-    /* Determine the code generator flags */
-    flags = TypeOf (lval->e_tptr) | CF_FORCECHAR;
+    /* Initialize the code generator flags */
+    lflags = 0;
+    rflags = 0;
 
     /* Evaluate the rhs */
     if (evalexpr (CF_NONE, hie1, &lval2) == 0) {
-       /* The resulting value is a constant. */
+       /* The resulting value is a constant. */
                if (MustScale) {
-           /* lhs is a pointer, scale rhs */
-           lval2.e_const *= SizeOf (lval->e_tptr+1);
-       }
-       flags |= CF_CONST;
+           /* lhs is a pointer, scale rhs */
+           lval2.ConstVal *= CheckedSizeOf (lval->Type+1);
+       }
+       rflags |= CF_CONST;
+       lflags |= CF_CONST;
     } else {
-       /* rhs is not constant and already in the primary register */
+       /* rhs is not constant and already in the primary register */
                if (MustScale) {
-           /* lhs is a pointer, scale rhs */
-                   g_scale (TypeOf (lval2.e_tptr), SizeOf (lval->e_tptr+1));
-       }
+           /* lhs is a pointer, scale rhs */
+                   g_scale (TypeOf (lval2.Type), CheckedSizeOf (lval->Type+1));
+       }
     }
 
-    /* Adjust the rhs to the lhs */
-    g_typeadjust (flags, TypeOf (lval2.e_tptr));
+    /* Setup the code generator flags */
+    lflags |= TypeOf (lval->Type) | CF_FORCECHAR;
+    rflags |= TypeOf (lval2.Type);
+
+    /* Cast the rhs to the type of the lhs */
+    g_typecast (lflags, rflags);
 
     /* Output apropriate code */
-    if (lval->e_flags & E_MGLOBAL) {
+    if (lval->Flags & E_MGLOBAL) {
        /* Static variable */
-       flags |= GlobalModeFlags (lval->e_flags);
+       lflags |= GlobalModeFlags (lval->Flags);
        if (Gen->Tok == TOK_PLUS_ASSIGN) {
-           g_addeqstatic (flags, lval->e_name, lval->e_const, lval2.e_const);
+           g_addeqstatic (lflags, lval->Name, lval->ConstVal, lval2.ConstVal);
        } else {
-                   g_subeqstatic (flags, lval->e_name, lval->e_const, lval2.e_const);
+                   g_subeqstatic (lflags, lval->Name, lval->ConstVal, lval2.ConstVal);
        }
-    } else if (lval->e_flags & E_MLOCAL) {
+    } else if (lval->Flags & E_MLOCAL) {
        /* ref to localvar */
        if (Gen->Tok == TOK_PLUS_ASSIGN) {
-           g_addeqlocal (flags, lval->e_const, lval2.e_const);
+           g_addeqlocal (lflags, lval->ConstVal, lval2.ConstVal);
        } else {
-           g_subeqlocal (flags, lval->e_const, lval2.e_const);
+           g_subeqlocal (lflags, lval->ConstVal, lval2.ConstVal);
        }
-    } else if (lval->e_flags & E_MCONST) {
+    } else if (lval->Flags & E_MCONST) {
        /* ref to absolute address */
-       flags |= CF_ABSOLUTE;
+       lflags |= CF_ABSOLUTE;
        if (Gen->Tok == TOK_PLUS_ASSIGN) {
-           g_addeqstatic (flags, lval->e_const, 0, lval2.e_const);
+           g_addeqstatic (lflags, lval->ConstVal, 0, lval2.ConstVal);
        } else {
-                   g_subeqstatic (flags, lval->e_const, 0, lval2.e_const);
+                   g_subeqstatic (lflags, lval->ConstVal, 0, lval2.ConstVal);
        }
-    } else if (lval->e_flags & E_MEXPR) {
+    } else if (lval->Flags & E_MEXPR) {
                /* Address in a/x. */
        if (Gen->Tok == TOK_PLUS_ASSIGN) {
-                   g_addeqind (flags, lval->e_const, lval2.e_const);
+                   g_addeqind (lflags, lval->ConstVal, lval2.ConstVal);
        } else {
-                   g_subeqind (flags, lval->e_const, lval2.e_const);
+                   g_subeqind (lflags, lval->ConstVal, lval2.ConstVal);
        }
     } else {
                Internal ("Invalid addressing mode");
     }
 
     /* Expression is in the primary now */
-    lval->e_flags = E_MEXPR;
+    lval->Flags = E_MEXPR;
 }
 
 
 
-static void Assignment (struct expent* lval)
-/* Parse an assignment */
-{
-    int k;
-    struct expent lval2;
-    unsigned flags;
-    type* ltype = lval->e_tptr;
-
-    /* Check for assignment to const */
-    if (IsQualConst (ltype)) {
-       Error ("Assignment to const");
-    }
-
-    /* cc65 does not have full support for handling structs by value. Since
-     * assigning structs is one of the more useful operations from this
-     * familiy, allow it here.
-     */
-    if (IsClassStruct (ltype)) {
-
-               /* Bring the address of the lhs into the primary and push it */
-       exprhs (0, 0, lval);
-       g_push (CF_PTR | CF_UNSIGNED, 0);
-
-       /* Get the expression on the right of the '=' into the primary */
-       k = hie1 (&lval2);
-       if (k) {
-           /* Get the address */
-           exprhs (0, 0, &lval2);
-       } else {
-           /* We need an lvalue */
-           Error ("Invalid lvalue in assignment");
-       }
-
-       /* Push the address (or whatever is in ax in case of errors) */
-       g_push (CF_PTR | CF_UNSIGNED, 0);
-
-       /* Check for equality of the structs */
-       if (TypeCmp (ltype, lval2.e_tptr) < TC_EQUAL) {
-           Error ("Incompatible types");
-       }
-
-       /* Load the size of the struct into the primary */
-       g_getimmed (CF_INT | CF_UNSIGNED | CF_CONST, SizeOf (ltype), 0);
-
-       /* Call the memcpy function */
-       g_call (CF_FIXARGC, "memcpy", 4);
-
-    } else {
-
-       /* Get the address on stack if needed */
-       PushAddr (lval);
-
-       /* No struct, setup flags for the load */
-       flags = SizeOf (ltype) == 1? CF_FORCECHAR : CF_NONE;
-
-       /* Get the expression on the right of the '=' into the primary */
-       if (evalexpr (flags, hie1, &lval2) == 0) {
-           /* Constant expression. Adjust the types */
-           assignadjust (ltype, &lval2);
-           /* Put the value into the primary register */
-           lconst (flags, &lval2);
-       } else {
-           /* Expression is not constant and already in the primary */
-           assignadjust (ltype, &lval2);
-       }
-
-       /* Generate a store instruction */
-       store (lval);
-
-    }
-
-    /* Value is still in primary */
-    lval->e_flags = E_MEXPR;
-}
-
-
-
-int hie1 (struct expent* lval)
+int hie1 (ExprDesc* lval)
 /* Parse first level of expression hierarchy. */
 {
     int k;
 
     k = hieQuest (lval);
-    switch (curtok) {
+    switch (CurTok.Tok) {
 
        case TOK_RPAREN:
        case TOK_SEMI:
@@ -2888,13 +3023,13 @@ int hie1 (struct expent* lval)
 
 
 
-int hie0 (struct expent *lval)
+static int hie0 (ExprDesc *lval)
 /* Parse comma operator. */
 {
     int k;
 
     k = hie1 (lval);
-    while (curtok == TOK_COMMA) {
+    while (CurTok.Tok == TOK_COMMA) {
        NextToken ();
        k = hie1 (lval);
     }
@@ -2903,7 +3038,7 @@ int hie0 (struct expent *lval)
 
 
 
-int evalexpr (unsigned flags, int (*f) (struct expent*), struct expent* lval)
+int evalexpr (unsigned flags, int (*f) (ExprDesc*), ExprDesc* lval)
 /* Will evaluate an expression via the given function. If the result is a
  * constant, 0 is returned and the value is put in the lval struct. If the
  * result is not constant, exprhs is called to bring the value into the
@@ -2914,7 +3049,7 @@ int evalexpr (unsigned flags, int (*f) (struct expent*), struct expent* lval)
 
     /* Evaluate */
     k = f (lval);
-    if (k == 0 && lval->e_flags == E_MCONST) {
+    if (k == 0 && lval->Flags == E_MCONST) {
        /* Constant expression */
        return 0;
     } else {
@@ -2926,7 +3061,7 @@ int evalexpr (unsigned flags, int (*f) (struct expent*), struct expent* lval)
 
 
 
-int expr (int (*func) (struct expent*), struct expent *lval)
+static int expr (int (*func) (ExprDesc*), ExprDesc *lval)
 /* Expression parser; func is either hie0 or hie1. */
 {
     int k;
@@ -2949,127 +3084,127 @@ int expr (int (*func) (struct expent*), struct expent *lval)
 
 
 
-void expression1 (struct expent* lval)
+void expression1 (ExprDesc* lval)
 /* Evaluate an expression on level 1 (no comma operator) and put it into
  * the primary register
  */
 {
-    memset (lval, 0, sizeof (*lval));
+    InitExprDesc (lval);
     exprhs (CF_NONE, expr (hie1, lval), lval);
 }
 
 
 
-void expression (struct expent* lval)
+void expression (ExprDesc* lval)
 /* Evaluate an expression and put it into the primary register */
 {
-    memset (lval, 0, sizeof (*lval));
+    InitExprDesc (lval);
     exprhs (CF_NONE, expr (hie0, lval), lval);
 }
 
 
 
-void constexpr (struct expent* lval)
+void ConstExpr (ExprDesc* lval)
 /* Get a constant value */
 {
-    memset (lval, 0, sizeof (*lval));
-    if (expr (hie1, lval) != 0 || (lval->e_flags & E_MCONST) == 0) {
+    InitExprDesc (lval);
+    if (expr (hie1, lval) != 0 || (lval->Flags & E_MCONST) == 0) {
        Error ("Constant expression expected");
        /* To avoid any compiler errors, make the expression a valid const */
-       lval->e_flags = E_MCONST;
-       lval->e_tptr = type_int;
-       lval->e_const = 0;
+       MakeConstIntExpr (lval, 1);
     }
 }
 
 
 
-void intexpr (struct expent* lval)
-/* Get an integer expression */
+void ConstIntExpr (ExprDesc* Val)
+/* Get a constant int value */
 {
-    expression (lval);
-    if (!IsClassInt (lval->e_tptr)) {
-       Error ("Integer expression expected");
-       /* To avoid any compiler errors, make the expression a valid int */
-       lval->e_flags = E_MCONST;
-       lval->e_tptr = type_int;
-       lval->e_const = 0;
+    InitExprDesc (Val);
+    if (expr (hie1, Val) != 0        ||
+       (Val->Flags & E_MCONST) == 0 ||
+       !IsClassInt (Val->Type)) {
+       Error ("Constant integer expression expected");
+       /* To avoid any compiler errors, make the expression a valid const */
+       MakeConstIntExpr (Val, 1);
     }
 }
 
 
 
-void boolexpr (struct expent* lval)
-/* Get a boolean expression */
+void intexpr (ExprDesc* lval)
+/* Get an integer expression */
 {
-    /* Read an expression */
     expression (lval);
-
-    /* If it's an integer, it's ok. If it's not an integer, but a pointer,
-     * the pointer used in a boolean context is also ok
-     */
-    if (!IsClassInt (lval->e_tptr) && !IsClassPtr (lval->e_tptr)) {
-       Error ("Boolean expression expected");
-       /* To avoid any compiler errors, make the expression a valid int */
-       lval->e_flags = E_MCONST;
-       lval->e_tptr = type_int;
-       lval->e_const = 0;
+    if (!IsClassInt (lval->Type)) {
+       Error ("Integer expression expected");
+       /* To avoid any compiler errors, make the expression a valid int */
+       MakeConstIntExpr (lval, 1);
     }
 }
 
 
 
-void test (unsigned label, int cond)
-/* Generate code to perform test and jump if false. */
+void Test (unsigned Label, int Invert)
+/* Evaluate a boolean test expression and jump depending on the result of
+ * the test and on Invert.
+ */
 {
     int k;
-    struct expent lval;
+    ExprDesc lval;
 
-    /* Eat the parenthesis */
-    ConsumeLParen ();
+    /* Evaluate the expression */
+    k = expr (hie0, InitExprDesc (&lval));
+
+    /* Check for a boolean expression */
+    CheckBoolExpr (&lval);
 
-    /* Prepare the expression, setup labels */
-    memset (&lval, 0, sizeof (lval));
+    /* Check for a constant expression */
+    if (k == 0 && lval.Flags == E_MCONST) {
 
-    /* Generate code to eval the expr */
-    k = expr (hie0, &lval);
-    if (k == 0 && lval.e_flags == E_MCONST) {
        /* Constant rvalue */
-               if (cond == 0 && lval.e_const == 0) {
-           g_jump (label);
+               if (!Invert && lval.ConstVal == 0) {
+           g_jump (Label);
            Warning ("Unreachable code");
-       } else if (cond && lval.e_const) {
-           g_jump (label);
+       } else if (Invert && lval.ConstVal != 0) {
+           g_jump (Label);
        }
-       ConsumeRParen ();
-       return;
-    }
 
-    /* If the expr hasn't set condition codes, set the force-test flag */
-    if ((lval.e_test & E_CC) == 0) {
-       lval.e_test |= E_FORCETEST;
-    }
+    } else {
 
-    /* Load the value into the primary register */
-    exprhs (CF_FORCECHAR, k, &lval);
+        /* If the expr hasn't set condition codes, set the force-test flag */
+        if ((lval.Test & E_CC) == 0) {
+            lval.Test |= E_FORCETEST;
+        }
 
-    /* Check for the closing brace */
-    ConsumeRParen ();
+        /* Load the value into the primary register */
+        exprhs (CF_FORCECHAR, k, &lval);
 
-    /* Generate the jump */
-    if (cond) {
-       g_truejump (CF_NONE, label);
-    } else {
-       /* Special case (putting this here is a small hack - but hey, the
-        * compiler itself is one big hack...): If a semicolon follows, we
-        * don't have a statement and may omit the jump.
-        */
-       if (curtok != TOK_SEMI) {
-            g_falsejump (CF_NONE, label);
-       }
+        /* Generate the jump */
+        if (Invert) {
+            g_truejump (CF_NONE, Label);
+        } else {
+            g_falsejump (CF_NONE, Label);
+        }
     }
 }
 
 
 
+void TestInParens (unsigned Label, int Invert)
+/* Evaluate a boolean test expression in parenthesis and jump depending on
+ * the result of the test * and on Invert.
+ */
+{
+    /* Eat the parenthesis */
+    ConsumeLParen ();
+
+    /* Do the test */
+    Test (Label, Invert);
+
+    /* Check for the closing brace */
+    ConsumeRParen ();
+}
+
+