* src/pic16/glue.c, src/SDCCast.c, src/SDCCast.h, src/SDCCBBlock.c,
[fw/sdcc] / src / pic16 / glue.c
index f59f1536014df40d7ed4d53c53e100e494ec9ad3..63ee1c71c37bd65376759e9f348439ddb0ef18f1 100644 (file)
 #include "gen.h"
 #include "device.h"
 #include "main.h"
+#include "dbuf_string.h"
 #include <string.h>
 
-#include <string.h>
-
-
-#ifdef WORDS_BIGENDIAN
-  #define _ENDIAN(x)  (3-x)
-#else
-  #define _ENDIAN(x)  (x)
-#endif
-
-#define BYTE_IN_LONG(x,b) ((x>>(8*_ENDIAN(b)))&0xff)
 
 extern symbol *interrupts[256];
 void pic16_printIval (symbol * sym, sym_link * type, initList * ilist, char ptype, void *p);
@@ -49,18 +40,17 @@ extern int noAlloc;
 extern set *publics;
 extern set *externs;
 extern unsigned maxInterrupts;
-extern int maxRegBank;
 extern symbol *mainf;
 extern char *VersionString;
-extern FILE *codeOutFile;
-extern set *tmpfileSet;
-extern set *tmpfileNameSet;
+extern struct dbuf_s *codeOutBuf;
 extern char *iComments1;
 extern char *iComments2;
-//extern void emitStaticSeg (memmap * map);
 
 extern int initsfpnt;
+extern unsigned long pFile_isize;
 
+extern unsigned long pic16_countInstructions();
+set *pic16_localFunctions = NULL;
 set *rel_idataSymSet=NULL;
 set *fix_idataSymSet=NULL;
 
@@ -68,7 +58,8 @@ extern DEFSETFUNC (closeTmpFiles);
 extern DEFSETFUNC (rmTmpFiles);
 
 extern void pic16_AnalyzeBanking (void);
-extern void copyFile (FILE * dest, FILE * src);
+extern void pic16_OptimizeJumps ();
+extern void pic16_OptimizeBanksel ();
 extern void pic16_InlinepCode(void);
 extern void pic16_writeUsedRegs(FILE *);
 
@@ -76,15 +67,14 @@ extern void initialComments (FILE * afile);
 extern void printPublics (FILE * afile);
 
 void  pic16_pCodeInitRegisters(void);
-pCodeOp *pic16_popGetLit(unsigned int lit);
-pCodeOp *pic16_popGetLit2(unsigned int lit, pCodeOp *arg2);
 pCodeOp *pic16_popCopyReg(pCodeOpReg *pc);
 extern void pic16_pCodeConstString(char *name, char *value);
 
+
 /*-----------------------------------------------------------------*/
 /* aopLiteral - string from a literal value                        */
 /*-----------------------------------------------------------------*/
-int pic16aopLiteral (value *val, int offset)
+unsigned int pic16aopLiteral (value *val, int offset)
 {
   union {
     float f;
@@ -93,8 +83,14 @@ int pic16aopLiteral (value *val, int offset)
 
   /* if it is a float then it gets tricky */
   /* otherwise it is fairly simple */
-  if (!IS_FLOAT(val->type)) {
-    unsigned long v = (unsigned long) floatFromVal(val);
+  if (!(IS_FLOAT(val->type) || IS_FIXED(val->type))) {
+    unsigned long v = ulFromVal (val);
+
+    return ( (v >> (offset * 8)) & 0xff);
+  }
+
+  if(IS_FIXED16X16(val->type)) {
+    unsigned long v = (unsigned long)fixed16x16FromDouble( floatFromVal( val ) );
 
     return ( (v >> (offset * 8)) & 0xff);
   }
@@ -123,267 +119,255 @@ pic16emitRegularMap (memmap * map, bool addPublics, bool arFlag)
   symbol *sym;
 //  int i, size, bitvars = 0;;
 
-//     fprintf(stderr, "%s:%d map name= %s\n", __FUNCTION__, __LINE__, map->sname);
-       
-       if(addPublics)
-               fprintf (map->oFile, ";\t.area\t%s\n", map->sname);
-               /* print the area name */
+//      fprintf(stderr, "%s:%d map name= %s\n", __FUNCTION__, __LINE__, map->sname);
+
+        if(addPublics)
+                dbuf_printf (&map->oBuf, ";\t.area\t%s\n", map->sname);
+                /* print the area name */
 
-       for (sym = setFirstItem (map->syms); sym; sym = setNextItem (map->syms)) {
+        for (sym = setFirstItem (map->syms); sym; sym = setNextItem (map->syms)) {
 
 #if 0
-               fprintf(stderr, "%s\t%s: sym: %s\tused: %d\textern: %d\tstatic: %d\taggregate: %d\n",
-                       __FUNCTION__,
-                       map->sname, sym->name, sym->used, IS_EXTERN(sym->etype), IS_STATIC(sym->etype),
-                       IS_AGGREGATE(sym->type));
-               printTypeChain( sym->type, stderr );
-               fprintf(stderr, "\n");
+                fprintf(stderr, "%s\t%s: sym: %s\tused: %d\textern: %d\tstatic: %d\taggregate: %d\tregister: 0x%x\tfunction: %d\n",
+                        __FUNCTION__,
+                        map->sname, sym->name, sym->used, IS_EXTERN(sym->etype), IS_STATIC(sym->etype),
+                        IS_AGGREGATE(sym->type), (SPEC_SCLS(sym->etype) == S_REGISTER), IS_FUNC(sym->type));
+                printTypeChain( sym->type, stderr );
+                fprintf(stderr, "\n");
 #endif
 
-               /* if extern then add to externs */
-               if (IS_EXTERN (sym->etype)) {
-                       
-                       /* reduce overhead while linking by not declaring
-                        * extern unused external functions (usually declared
-                        * in header files) */
-                       if(IS_FUNC(sym->type) && !sym->used)continue;
-                       
-                       checkAddSym(&externs, sym);
-                       continue;
-               }
-               
-               /* if allocation required check is needed
-                *  then check if the symbol really requires
-                * allocation only for local variables */
-                if (arFlag && !IS_AGGREGATE (sym->type) &&
-                       !(sym->_isparm && !IS_REGPARM (sym->etype)) &&
-                       !sym->allocreq && sym->level) {
-
-//                     fprintf(stderr, "%s:%d special case, continuing...\n", __FILE__, __LINE__);
-
-                       continue;
-               }
-
-               /* if global variable & not static or extern
-                * and addPublics allowed then add it to the public set */
-               if ((sym->used) && (sym->level == 0 ||
-                       (sym->_isparm && !IS_REGPARM (sym->etype))) &&
-                       addPublics &&
-                       !IS_STATIC (sym->etype) && !IS_FUNC(sym->type)) {
-                 
-                       checkAddSym(&publics, sym);
-               } else
-                       if(IS_STATIC(sym->etype)
-                               && !(sym->ival && !sym->level)
-                       ) {
-                         regs *reg;
-                               /* add it to udata list */
-
-//                             fprintf(stderr, "%s:%d adding %s (%s) remat=%d\n", __FILE__, __LINE__,
-//                                     sym->name, sym->rname, sym->remat);
-                                       
-                                               //, OP_SYMBOL(operandFromSymbol(sym))->name);
-#define SET_IMPLICIT   1
+                /* if extern then add to externs */
+                if (IS_EXTERN (sym->etype)) {
+                        /* reduce overhead while linking by not declaring
+                         * extern unused external functions (usually declared
+                         * in header files) */
+                        if(IS_FUNC(sym->type) && !sym->used)continue;
+
+                        /* make sure symbol is not in publics section */
+                        if(!checkSym(publics, sym))
+                                checkAddSym(&externs, sym);
+                        continue;
+                }
+
+                /* if allocation required check is needed
+                 * then check if the symbol really requires
+                 * allocation only for local variables */
+                 if (arFlag && !IS_AGGREGATE (sym->type) &&
+                        !(sym->_isparm && !IS_REGPARM (sym->etype)) &&
+                        !sym->allocreq && sym->level) {
+
+//                      fprintf(stderr, "%s:%d special case, continuing...\n", __FILE__, __LINE__);
+
+                        continue;
+                }
+
+                /* if global variable & not static or extern
+                 * and addPublics allowed then add it to the public set */
+                if ((sym->used) && (sym->level == 0 ||
+                        (sym->_isparm && !IS_REGPARM (sym->etype))) &&
+                        addPublics &&
+                        !IS_STATIC (sym->etype) && !IS_FUNC(sym->type)) {
+
+                        checkAddSym(&publics, sym);
+                } else
+                        /* new version */
+                        if(IS_STATIC(sym->etype)
+                                && !sym->ival) /* && !sym->level*/ {
+                          regs *reg;
+                          sectSym *ssym;
+                          int found=0;
+
+//                            debugf("adding symbol %s\n", sym->name);
+#define SET_IMPLICIT    1
 
 #if SET_IMPLICIT
-                               if(IS_STRUCT(sym->type))
-                                       sym->implicit = 1;
+                                if(IS_STRUCT(sym->type))
+                                        sym->implicit = 1;
 #endif
 
-                               reg = pic16_allocDirReg( operandFromSymbol( sym ));
-                               
-                               {
-                                 sectSym *ssym;
-                                 int found=0;
-                                 
-#if 1
-                                       for(ssym=setFirstItem(sectSyms); ssym; ssym=setNextItem(sectSyms)) {
-                                               if(!strcmp(ssym->name, reg->name))found=1;
-                                       }
-#endif
-
-                                       if(!found)checkAddReg(&pic16_rel_udata, reg);
-                               }
-                       }
+                                reg = pic16_allocDirReg( operandFromSymbol( sym ));
 
-               /* if extern then do nothing or is a function
-                * then do nothing */
-               if (IS_FUNC (sym->type) && !IS_STATIC(sym->etype)) {
-                       if(SPEC_OCLS(sym->etype) == code) {
-//                             fprintf(stderr, "%s:%d: symbol added: %s\n", __FILE__, __LINE__, sym->rname);
-                               checkAddSym(&publics, sym);
-                       }
-                       continue;
-               }
+                                if(reg) {
+                                  for(ssym=setFirstItem(sectSyms); ssym; ssym=setNextItem(sectSyms)) {
+                                    if(!strcmp(ssym->name, reg->name))found=1;
+                                  }
 
+                                  if(!found)
+                                    checkAddReg(&pic16_rel_udata, reg);
 #if 0
-               /* print extra debug info if required */
-               if (options.debug || sym->level == 0) {
-                       cdbWriteSymbol (sym);   //, cdbFile, FALSE, FALSE);
-
-                       if (!sym->level)        /* global */
-                               if (IS_STATIC (sym->etype))
-                                       fprintf (map->oFile, "F%s_", moduleName);               /* scope is file */
-                               else
-                                       fprintf (map->oFile, "G_");     /* scope is global */
-                       else
-                               /* symbol is local */
-                               fprintf (map->oFile, "L%s_", (sym->localof ? sym->localof->name : "-null-"));
-                       fprintf (map->oFile, "%s_%d_%d", sym->name, sym->level, sym->block);
-               }
+                                  else
+                                    debugf("Did find %s in pic16_rel_udata already. Check!\n", reg->name);
+//                                  checkAddSym(&publics, sym);
 #endif
 
-
-               /* if is has an absolute address then generate
-               an equate for this no need to allocate space */
-               if (SPEC_ABSA (sym->etype)) {
-//                     if (options.debug || sym->level == 0)
-//                             fprintf (stderr,"; %s == 0x%04x\t\treqv= %p nRegs= %d\n",
-//                                     sym->name, SPEC_ADDR (sym->etype), sym->reqv, sym->regType);
-
-                       fprintf (map->oFile, "%s\tEQU\t0x%04x\n",
-                               sym->rname,
-                               SPEC_ADDR (sym->etype));
-
-                       /* emit only if it is global */
-                       if(sym->level == 0) {
-                         regs *reg;
-
-                               reg = pic16_dirregWithName( sym->name );
-                               if(!reg) {
-                                       /* here */
-//                                     fprintf(stderr, "%s:%d: implicit add of symbol = %s\n",
-//                                                     __FUNCTION__, __LINE__, sym->name);
-
-                                       /* if IS_STRUCT is omitted the following
-                                        * fixes structures but break char/int etc */
+                                }
+                        }
+
+                /* if extern then do nothing or is a function
+                 * then do nothing */
+                if (IS_FUNC (sym->type) && !IS_STATIC(sym->etype)) {
+                        if(SPEC_OCLS(sym->etype) == code) {
+//                              fprintf(stderr, "%s:%d: symbol added: %s\n", __FILE__, __LINE__, sym->rname);
+                                checkAddSym(&publics, sym);
+                        }
+                        continue;
+                }
+
+                /* if is has an absolute address then generate
+                an equate for this no need to allocate space */
+                if (SPEC_ABSA (sym->etype)) {
+//                              fprintf (stderr,"; %s == 0x%04x\t\treqv= %p nRegs= %d\n",
+//                                      sym->name, SPEC_ADDR (sym->etype), sym->reqv, sym->regType);
+
+                        dbuf_printf (&map->oBuf, "%s\tEQU\t0x%04x\n",
+                                sym->rname,
+                                SPEC_ADDR (sym->etype));
+
+                        /* emit only if it is global */
+                        if(sym->level == 0) {
+                          regs *reg;
+
+                                reg = pic16_dirregWithName( sym->name );
+                                if(!reg) {
+                                        /* here */
+//                                      fprintf(stderr, "%s:%d: implicit add of symbol = %s\n",
+//                                                      __FUNCTION__, __LINE__, sym->name);
+
+                                        /* if IS_STRUCT is omitted the following
+                                         * fixes structures but break char/int etc */
 #if SET_IMPLICIT
-                                       if(IS_STRUCT(sym->type))
-                                               sym->implicit = 1;              // mark as implicit
+                                        if(IS_STRUCT(sym->type))
+                                                sym->implicit = 1;              // mark as implicit
 #endif
-                                       if(!sym->ival) {
-                                               reg = pic16_allocDirReg( operandFromSymbol(sym) );
-                                               if(reg) {
-                                                       if(checkAddReg(&pic16_fix_udata, reg)) {
-                                                               /* and add to globals list if not exist */
-                                                               addSet(&publics, sym);
-                                                       }
-                                               }
-                                       } else
-                                               addSet(&publics, sym);
-                               }
-                       }
-               } else {
-                       if(!sym->used && (sym->level == 0)) {
-                         regs *reg;
-
-                               /* symbol not used, just declared probably, but its in
-                                * level 0, so we must declare it fine as global */
-                               
-//                             fprintf(stderr, "EXTRA symbol declaration sym= %s\n", sym->name);
+                                        if(!sym->ival) {
+                                                reg = pic16_allocDirReg( operandFromSymbol(sym) );
+                                                if(reg) {
+                                                        if(checkAddReg(&pic16_fix_udata, reg)) {
+                                                                /* and add to globals list if not exist */
+                                                                addSet(&publics, sym);
+                                                        }
+                                                }
+                                        } else
+                                                addSet(&publics, sym);
+                                }
+                        }
+                } else {
+                        if(!sym->used && (sym->level == 0)) {
+                          regs *reg;
+
+                                /* symbol not used, just declared probably, but its in
+                                 * level 0, so we must declare it fine as global */
+
+//                              fprintf(stderr, "EXTRA symbol declaration sym= %s\n", sym->name);
 
 #if SET_IMPLICIT
-                               if(IS_STRUCT(sym->type))
-                                       sym->implicit = 1;              // mark as implicit
+                                if(IS_STRUCT(sym->type))
+                                        sym->implicit = 1;              // mark as implicit
 #endif
-                               if(!sym->ival) {
-                                       if(IS_AGGREGATE(sym->type)) {
-                                               reg=pic16_allocRegByName(sym->rname, getSize( sym->type ), NULL);
-                                       } else {
-                                               reg = pic16_allocDirReg( operandFromSymbol( sym ) );
-                                       }
-
-                                       {
-                                         sectSym *ssym;
-                                         int found=0;
-                                 
-#if 1
-                                               for(ssym=setFirstItem(sectSyms); ssym; ssym=setNextItem(sectSyms)) {
-                                                       if(!strcmp(ssym->name, reg->name))found=1;
-                                               }
+                                if(!sym->ival) {
+                                        if(IS_AGGREGATE(sym->type)) {
+                                                reg=pic16_allocRegByName(sym->rname, getSize( sym->type ), NULL);
+                                        } else {
+                                                reg = pic16_allocDirReg( operandFromSymbol( sym ) );
+                                        }
+
+                                        {
+                                          sectSym *ssym;
+                                          int found=0;
+
+#if 0
+                                                fprintf(stderr, "%s:%d sym->rname: %s reg: %p reg->name: %s\n", __FILE__, __LINE__,
+                                                        sym->rname, reg, (reg?reg->name:"<<NULL>>"));
 #endif
 
-                                               if(!found)
-                                                       if(checkAddReg(&pic16_rel_udata, reg)) {
-                                                               addSetHead(&publics, sym);
-                                                       }
+                                                if(reg) {
+                                                  for(ssym=setFirstItem(sectSyms); ssym; ssym=setNextItem(sectSyms)) {
+                                                    if(!strcmp(ssym->name, reg->name))found=1;
+                                                  }
+
+                                                  if(!found)
+                                                    if(checkAddReg(&pic16_rel_udata, reg)) {
+                                                      addSetHead(&publics, sym);
+                                                    }
+                                                }
+                                        }
 
-                                       }
 
 
-               
-                               } else
+                                } else
 
-                                       addSetHead(&publics, sym);
-                       }
+                                        addSetHead(&publics, sym);
+                        }
 
 #if 0
-                       /* allocate space */
-                       /* If this is a bit variable, then allocate storage after 8 bits have been declared */
-                       /* unlike the 8051, the pic does not have a separate bit area. So we emulate bit ram */
-                       /* by grouping the bits together into groups of 8 and storing them in the normal ram. */
-                       if (IS_BITVAR (sym->etype)) {
-                               bitvars++;
-                       } else {
-                               fprintf (map->oFile, "\t%s\n", sym->rname);
-                               if ((size = (unsigned int) getSize (sym->type) & 0xffff) > 1) {
-                                       for (i = 1; i < size; i++)
-                                               fprintf (map->oFile, "\t%s_%d\n", sym->rname, i);
-                               }
-                       }
-                       fprintf (map->oFile, "\t.ds\t0x%04x\n", (unsigned int)getSize (sym->type) & 0xffff);
+                        /* allocate space */
+                        /* If this is a bit variable, then allocate storage after 8 bits have been declared */
+                        /* unlike the 8051, the pic does not have a separate bit area. So we emulate bit ram */
+                        /* by grouping the bits together into groups of 8 and storing them in the normal ram. */
+                        if (IS_BITVAR (sym->etype)) {
+                                bitvars++;
+                        } else {
+                                dbuf_printf (map->oBuf, "\t%s\n", sym->rname);
+                                if ((size = (unsigned int) getSize (sym->type) & 0xffff) > 1) {
+                                        for (i = 1; i < size; i++)
+                                                dbuf_printf (map->oBuf, "\t%s_%d\n", sym->rname, i);
+                                }
+                        }
+                        dbuf_printf (map->oBuf, "\t.ds\t0x%04x\n", (unsigned int)getSize (sym->type) & 0xffff);
 #endif
-               }
-       
-               /* FIXME -- VR Fix the following, so that syms to be placed
-                * in the idata section and let linker decide about their fate */
+                }
+
+                /* FIXME -- VR Fix the following, so that syms to be placed
+                 * in the idata section and let linker decide about their fate */
 
-               /* if it has an initial value then do it only if
-                       it is a global variable */
+                /* if it has an initial value then do it only if
+                        it is a global variable */
 
-               if (sym->ival && sym->level == 0) {
-                 ast *ival = NULL;
+                if (sym->ival
+                  && ((sym->level == 0)
+                      || IS_STATIC(sym->etype)) ) {
+                  ast *ival = NULL;
 
 #if 0
-                       if(SPEC_OCLS(sym->etype)==data) {
-                               fprintf(stderr, "%s: sym %s placed in data segment\n", map->sname, sym->name);
-                       }
+                        if(SPEC_OCLS(sym->etype)==data) {
+                                fprintf(stderr, "%s: sym %s placed in data segment\n", map->sname, sym->name);
+                        }
 
-                       if(SPEC_OCLS(sym->etype)==code) {
-                               fprintf(stderr, "%s: sym %s placed in code segment\n", map->sname, sym->name);
-                       }
+                        if(SPEC_OCLS(sym->etype)==code) {
+                                fprintf(stderr, "%s: sym %s placed in code segment\n", map->sname, sym->name);
+                        }
 #endif
 
 #if 0
-                       fprintf(stderr, "'%s': sym '%s' has initial value SPEC_ABSA: %d, IS_AGGREGATE: %d\n",
-                               map->sname, sym->name, SPEC_ABSA(sym->etype), IS_AGGREGATE(sym->type));
+                        fprintf(stderr, "'%s': sym '%s' has initial value SPEC_ABSA: %d, IS_AGGREGATE: %d\n",
+                                map->sname, sym->name, SPEC_ABSA(sym->etype), IS_AGGREGATE(sym->type));
 #endif
 
-                       if (IS_AGGREGATE (sym->type)) {
-                               if(SPEC_ABSA(sym->etype))
-                                       addSet(&fix_idataSymSet, copySymbol(sym));
-                               else
-                                       addSet(&rel_idataSymSet, copySymbol(sym));
-//                             ival = initAggregates (sym, sym->ival, NULL);
-                       } else {
-                               if(SPEC_ABSA(sym->etype))
-                                       addSet(&fix_idataSymSet, copySymbol(sym));
-                               else
-                                       addSet(&rel_idataSymSet, copySymbol(sym));
-
-//                                     ival = newNode ('=', newAst_VALUE(symbolVal (sym)),
-//                                             decorateType (resolveSymbols (list2expr (sym->ival)), RESULT_CHECK));
-                       }
-
-                       if(ival) {
-                               setAstLineno(ival, sym->lineDef);
-                               codeOutFile = statsg->oFile;
-                               GcurMemmap = statsg;
-                               eBBlockFromiCode (iCodeFromAst (ival));
-                               sym->ival = NULL;
-                       }
-               }
-       }
+                        if (IS_AGGREGATE (sym->type)) {
+                                if(SPEC_ABSA(sym->etype))
+                                        addSet(&fix_idataSymSet, copySymbol(sym));
+                                else
+                                        addSet(&rel_idataSymSet, copySymbol(sym));
+//                              ival = initAggregates (sym, sym->ival, NULL);
+                        } else {
+                                if(SPEC_ABSA(sym->etype))
+                                        addSet(&fix_idataSymSet, copySymbol(sym));
+                                else
+                                        addSet(&rel_idataSymSet, copySymbol(sym));
+
+//                                      ival = newNode ('=', newAst_VALUE(symbolVal (sym)),
+//                                              decorateType (resolveSymbols (list2expr (sym->ival)), RESULT_TYPE_NONE));
+                        }
+
+                        if(ival) {
+                                setAstFileLine (ival, sym->fileDef, sym->lineDef);
+                                codeOutBuf = &statsg->oBuf;
+                                GcurMemmap = statsg;
+                                eBBlockFromiCode (iCodeFromAst (ival));
+                                sym->ival = NULL;
+                        }
+                }
+        }
 }
 
 
@@ -401,14 +385,14 @@ value *pic16_initPointer (initList * ilist, sym_link *toType)
 
   expr = decorateType(resolveSymbols( list2expr (ilist) ), FALSE);
 //  expr = list2expr( ilist );
-  
+
   if (!expr)
     goto wrong;
-  
+
   /* try it the old way first */
-  if ((val = constExprValue (expr, FALSE)))
+  if (expr->etype && (val = constExprValue (expr, FALSE)))
     return val;
-  
+
   /* ( ptr + constant ) */
   if (IS_AST_OP (expr) &&
       (expr->opval.op == '+' || expr->opval.op == '-') &&
@@ -420,7 +404,7 @@ value *pic16_initPointer (initList * ilist, sym_link *toType)
                            expr->right,
                            expr->opval.op);
   }
-  
+
   /* (char *)&a */
   if (IS_AST_OP(expr) && expr->opval.op==CAST &&
       IS_AST_OP(expr->right) && expr->right->opval.op=='&') {
@@ -437,8 +421,8 @@ value *pic16_initPointer (initList * ilist, sym_link *toType)
      a variable or address of an array element */
   if (IS_AST_OP (expr) && expr->opval.op == '&') {
     /* address of symbol */
-    if (IS_AST_SYM_VALUE (expr->left)) {
-      val = copyValue (AST_VALUE (expr->left));
+    if (IS_AST_SYM_VALUE (expr->left) && expr->left->etype) {
+      val = AST_VALUE (expr->left);
       val->type = newLink (DECLARATOR);
       if(SPEC_SCLS (expr->left->etype) == S_CODE) {
         DCL_TYPE (val->type) = CPOINTER;
@@ -536,10 +520,10 @@ void _pic16_printPointerType (const char *name, char ptype, void *p)
 {
   char buf[256];
 
-       sprintf(buf, "LOW(%s)", name);
-       pic16_emitDS(buf, ptype, p);
-       sprintf(buf, "HIGH(%s)", name);
-       pic16_emitDS(buf, ptype, p);
+  sprintf (buf, "LOW(%s)", name);
+  pic16_emitDS (buf, ptype, p);
+  sprintf (buf, "HIGH(%s)", name);
+  pic16_emitDS (buf, ptype, p);
 }
 
 /*-----------------------------------------------------------------*/
@@ -548,40 +532,65 @@ void _pic16_printPointerType (const char *name, char ptype, void *p)
 void pic16_printPointerType (const char *name, char ptype, void *p)
 {
   _pic16_printPointerType (name, ptype, p);
-  pic16_flushDB(ptype, p);
+  //pic16_flushDB(ptype, p); /* breaks char* const arr[] = {&c, &c, &c}; */
 }
 
 /*-----------------------------------------------------------------*/
 /* printGPointerType - generates ival for generic pointer type     */
 /*-----------------------------------------------------------------*/
-void pic16_printGPointerType (const char *iname, const char *oname, const unsigned int itype,
-                   const unsigned int type, char ptype, void *p)
+void pic16_printGPointerType (const char *iname, const unsigned int itype,
+  char ptype, void *p)
 {
-  _pic16_printPointerType (iname, ptype, p);
+  char buf[256];
 
-  if(itype == FPOINTER || itype == CPOINTER) { // || itype == GPOINTER) {
-    char buf[256];
+  _pic16_printPointerType (iname, ptype, p);
 
-       sprintf(buf, "UPPER(%s)", iname);
-       pic16_emitDS(buf, ptype, p);
-  }
+  switch (itype)
+    {
+    case CPOINTER:
+    case FUNCTION:
+      sprintf (buf, "UPPER(%s)", iname);
+      pic16_emitDS (buf, ptype, p);
+      break;
+
+    case GPOINTER:
+    case POINTER:
+      sprintf (buf, "0x80");
+      pic16_emitDS (buf, ptype, p);
+      break;
+
+    /*
+     * FPOINTER and IPOINTER are not used in pic16 port
+      case FPOINTER:
+      case IPOINTER:
+     */
+    default:
+      debugf ("itype = %d\n", itype );
+      assert (0);
+    }
 
-  pic16_flushDB(ptype, p);
+  //pic16_flushDB(ptype, p); /* might break char* const arr[] = {...}; */
 }
 
 
+/* set to 0 to disable debug messages */
+#define DEBUG_PRINTIVAL 0
 
 /*-----------------------------------------------------------------*/
 /* pic16_printIvalType - generates ival for int/char               */
 /*-----------------------------------------------------------------*/
-static void 
+static void
 pic16_printIvalType (symbol *sym, sym_link * type, initList * ilist, char ptype, void *p)
 {
   value *val;
-  unsigned long ulval;
 
 //  fprintf(stderr, "%s for symbol %s\n",__FUNCTION__, sym->rname);
 
+#if DEBUG_PRINTIVAL
+  fprintf(stderr, "%s\n",__FUNCTION__);
+#endif
+
+
   /* if initList is deep */
   if (ilist && ilist->type == INIT_DEEP)
     ilist = ilist->init.deep;
@@ -592,33 +601,31 @@ pic16_printIvalType (symbol *sym, sym_link * type, initList * ilist, char ptype,
 
   if (!(val = list2val (ilist))) {
     // assuming a warning has been thrown
-    val=constVal("0");
+    val = constCharVal (0);
   }
 
   if (val->type != type) {
     val = valCastLiteral(type, floatFromVal(val));
   }
 
-  if(val) 
-    ulval = (unsigned long) floatFromVal (val);
-  else
-    ulval =0;
-
   switch (getSize (type)) {
   case 1:
-    pic16_emitDB(BYTE_IN_LONG(ulval,0), ptype, p);
+    pic16_emitDB(pic16aopLiteral(val, 0), ptype, p);
     break;
 
   case 2:
-    pic16_emitDB(BYTE_IN_LONG(ulval,0), ptype, p);
-    pic16_emitDB(BYTE_IN_LONG(ulval,1), ptype, p);
+    pic16_emitDB(pic16aopLiteral(val, 0), ptype, p);
+    pic16_emitDB(pic16aopLiteral(val, 1), ptype, p);
     break;
-
+  case 3:
+    pic16_emitDB(pic16aopLiteral(val, 0), ptype, p);
+    pic16_emitDB(pic16aopLiteral(val, 1), ptype, p);
+    pic16_emitDB(pic16aopLiteral(val, 2), ptype, p);
   case 4:
-    pic16_emitDB(BYTE_IN_LONG(ulval,0), ptype, p);
-    pic16_emitDB(BYTE_IN_LONG(ulval,1), ptype, p);
-    pic16_emitDB(BYTE_IN_LONG(ulval,2), ptype, p);
-    pic16_emitDB(BYTE_IN_LONG(ulval,3), ptype, p);
+    pic16_emitDB(pic16aopLiteral(val, 0), ptype, p);
+    pic16_emitDB(pic16aopLiteral(val, 1), ptype, p);
+    pic16_emitDB(pic16aopLiteral(val, 2), ptype, p);
+    pic16_emitDB(pic16aopLiteral(val, 3), ptype, p);
     break;
   }
 }
@@ -626,41 +633,64 @@ pic16_printIvalType (symbol *sym, sym_link * type, initList * ilist, char ptype,
 /*--------------------------------------------------------------------*/
 /* pic16_printIvalChar - generates initital value for character array */
 /*--------------------------------------------------------------------*/
-static int 
-pic16_printIvalChar (sym_link * type, initList * ilist, char *s, char ptype, void *p)
+static int
+pic16_printIvalChar (symbol *sym, sym_link * type, initList * ilist, char *s, char ptype, void *p)
 {
   value *val;
-  int remain;
+  int remain, len, ilen;
 
   if(!p)
     return 0;
 
+#if DEBUG_PRINTIVAL
+  fprintf(stderr, "%s\n",__FUNCTION__);
+#endif
 
-  // fprintf(stderr, "%s\n",__FUNCTION__);
-  if (!s)
-    {
+  if(!s) {
+    val = list2val (ilist);
 
-      val = list2val (ilist);
-      /* if the value is a character string  */
-      if (IS_ARRAY (val->type) && IS_CHAR (val->etype))
-       {
-         if (!DCL_ELEM (type))
-           DCL_ELEM (type) = strlen (SPEC_CVAL (val->etype).v_char) + 1;
-
-           for(remain=0; remain<DCL_ELEM(type); remain++)
-               pic16_emitDB(SPEC_CVAL(val->etype).v_char[ remain ], ptype, p);
-                       
-         if ((remain = (DCL_ELEM (type) - strlen (SPEC_CVAL (val->etype).v_char) - 1)) > 0) {
-             while (remain--) {
-                  pic16_emitDB(0x00, ptype, p);
-              }
+    /* if the value is a character string  */
+    if(IS_ARRAY (val->type) && IS_CHAR (val->etype)) {
+      /* length of initializer string (might contain \0, so do not use strlen) */
+      ilen = DCL_ELEM(val->type);
+
+      if(!DCL_ELEM (type))
+        DCL_ELEM (type) = ilen;
+
+      /* len is 0 if declartion equals initializer,
+       * >0 if declaration greater than initializer
+       * <0 if declaration less than initializer
+       * Strategy: if >0 emit 0x00 for the rest of the length,
+       * if <0 then emit only the length of declaration elements
+       * and warn user
+       */
+      len = DCL_ELEM (type) - ilen;
+
+//      fprintf(stderr, "%s:%d ilen = %i len = %i DCL_ELEM(type) = %i SPEC_CVAL-len = %i\n", __FILE__, __LINE__,
+//        ilen, len, DCL_ELEM(type), strlen(SPEC_CVAL(val->etype).v_char));
+
+      if(len >= 0) {
+        /* emit initializer */
+        for(remain=0; remain<ilen; remain++)
+          pic16_emitDB(SPEC_CVAL(val->etype).v_char[ remain ], ptype, p);
+
+          /* fill array with 0x00 */
+          while(len--) {
+            pic16_emitDB(0x00, ptype, p);
           }
-         return 1;
-       }
-      else
-       return 0;
-    }
-  else {
+      } else {
+        werror (W_EXCESS_INITIALIZERS, "array of chars", sym->name, sym->lineDef);
+
+        for(remain=0; remain<DCL_ELEM (type); remain++)
+          pic16_emitDB(SPEC_CVAL(val->etype).v_char[ remain ], ptype, p);
+      }
+
+
+//      if((remain = (DCL_ELEM (type) - strlen (SPEC_CVAL (val->etype).v_char) - 1)) > 0) {
+//      }
+      return 1;
+    } else return 0;
+  } else {
     for(remain=0; remain<strlen(s); remain++) {
         pic16_emitDB(s[remain], ptype, p);
     }
@@ -671,9 +701,9 @@ pic16_printIvalChar (sym_link * type, initList * ilist, char *s, char ptype, voi
 /*-----------------------------------------------------------------*/
 /* pic16_printIvalArray - generates code for array initialization        */
 /*-----------------------------------------------------------------*/
-static void 
+static void
 pic16_printIvalArray (symbol * sym, sym_link * type, initList * ilist,
-               char ptype, void *p)
+                char ptype, void *p)
 {
   initList *iloop;
   int lcnt = 0, size = 0;
@@ -682,6 +712,9 @@ pic16_printIvalArray (symbol * sym, sym_link * type, initList * ilist,
     return;
 
 
+#if DEBUG_PRINTIVAL
+  fprintf(stderr, "%s\n",__FUNCTION__);
+#endif
   /* take care of the special   case  */
   /* array of characters can be init  */
   /* by a string                      */
@@ -691,9 +724,9 @@ pic16_printIvalArray (symbol * sym, sym_link * type, initList * ilist,
       return;
     }
 
-    if(pic16_printIvalChar (type,
-                      (ilist->type == INIT_DEEP ? ilist->init.deep : ilist),
-                      SPEC_CVAL (sym->etype).v_char, ptype, p))
+    if(pic16_printIvalChar (sym, type,
+                       (ilist->type == INIT_DEEP ? ilist->init.deep : ilist),
+                       SPEC_CVAL (sym->etype).v_char, ptype, p))
       return;
   }
   /* not the special case             */
@@ -716,16 +749,16 @@ pic16_printIvalArray (symbol * sym, sym_link * type, initList * ilist,
       /* if not array limits given & we */
       /* are out of initialisers then   */
       if (!DCL_ELEM (type) && !iloop)
-       break;
+        break;
 
       /* no of elements given and we    */
       /* have generated for all of them */
       if (!--lcnt) {
-       /* if initializers left */
-       if (iloop) {
-         werror (W_EXCESS_INITIALIZERS, "array", sym->name, sym->lineDef);
-       }
-       break;
+        /* if initializers left */
+        if (iloop) {
+          werror (W_EXCESS_INITIALIZERS, "array", sym->name, sym->lineDef);
+        }
+        break;
       }
     }
 
@@ -748,6 +781,11 @@ void pic16_printIvalBitFields(symbol **sym, initList **ilist, char ptype, void *
   int size =0;
 
 
+#if DEBUG_PRINTIVAL
+  fprintf(stderr, "%s\n",__FUNCTION__);
+#endif
+
+
   do {
     unsigned long i;
     val = list2val(lilist);
@@ -760,10 +798,11 @@ void pic16_printIvalBitFields(symbol **sym, initList **ilist, char ptype, void *
       size = ((SPEC_BLEN (lsym->etype) / 8) +
               (SPEC_BLEN (lsym->etype) % 8 ? 1 : 0));
     }
-    i = (unsigned long)floatFromVal(val);
+    i = ulFromVal (val);
     i <<= SPEC_BSTR (lsym->etype);
     ival |= i;
     if (! ( lsym->next &&
+          (lilist && lilist->next) &&
           (IS_BITFIELD(lsym->next->type)) &&
           (SPEC_BSTR(lsym->next->etype)))) break;
     lsym = lsym->next;
@@ -771,29 +810,29 @@ void pic16_printIvalBitFields(symbol **sym, initList **ilist, char ptype, void *
   } while (1);
   switch (size) {
   case 1:
-       pic16_emitDB(BYTE_IN_LONG(ival, 0), ptype, p);
+        pic16_emitDB(BYTE_IN_LONG(ival, 0), ptype, p);
         break;
 
   case 2:
-       pic16_emitDB(BYTE_IN_LONG(ival, 0), ptype, p);
-       pic16_emitDB(BYTE_IN_LONG(ival, 1), ptype, p);
-        break;
+        pic16_emitDB(BYTE_IN_LONG(ival, 0), ptype, p);
+        pic16_emitDB(BYTE_IN_LONG(ival, 1), ptype, p);
+    break;
 
   case 4: /* EEP: why is this db and not dw? */
-       pic16_emitDB(BYTE_IN_LONG(ival, 0), ptype, p);
-       pic16_emitDB(BYTE_IN_LONG(ival, 1), ptype, p);
-       pic16_emitDB(BYTE_IN_LONG(ival, 2), ptype, p);
-       pic16_emitDB(BYTE_IN_LONG(ival, 3), ptype, p);
-        break;
+        pic16_emitDB(BYTE_IN_LONG(ival, 0), ptype, p);
+        pic16_emitDB(BYTE_IN_LONG(ival, 1), ptype, p);
+        pic16_emitDB(BYTE_IN_LONG(ival, 2), ptype, p);
+        pic16_emitDB(BYTE_IN_LONG(ival, 3), ptype, p);
+    break;
   default:
-       /* VR - only 1,2,4 size long can be handled???? Why? */
-       fprintf(stderr, "%s:%d: unhandled case. Contact author.\n", __FILE__, __LINE__);
-       assert(0);
+        /* VR - only 1,2,4 size long can be handled???? Why? */
+        fprintf(stderr, "%s:%d: unhandled case. Contact author.\n", __FILE__, __LINE__);
+        assert(0);
   }
   *sym = lsym;
   *ilist = lilist;
 }
+
 
 /*-----------------------------------------------------------------*/
 /* printIvalStruct - generates initial value for structures        */
@@ -804,6 +843,11 @@ void pic16_printIvalStruct (symbol * sym, sym_link * type,
   symbol *sflds;
   initList *iloop = NULL;
 
+
+#if DEBUG_PRINTIVAL
+  fprintf(stderr, "%s\n",__FUNCTION__);
+#endif
+
   sflds = SPEC_STRUCT (type)->fields;
 
   if (ilist) {
@@ -815,7 +859,7 @@ void pic16_printIvalStruct (symbol * sym, sym_link * type,
     iloop = ilist->init.deep;
   }
 
-  for (; sflds; sflds = sflds->next, iloop = (iloop ? iloop->next : NULL)) {
+  for (; (sflds && iloop); sflds = sflds->next, iloop = (iloop ? iloop->next : NULL)) {
 //    fprintf(stderr, "%s:%d sflds: %p\tiloop = %p\n", __FILE__, __LINE__, sflds, iloop);
     if (IS_BITFIELD(sflds->type)) {
       pic16_printIvalBitFields(&sflds, &iloop, ptype, p);
@@ -829,6 +873,55 @@ void pic16_printIvalStruct (symbol * sym, sym_link * type,
   return;
 }
 
+/*-----------------------------------------------------------------*/
+/* printIvalUnion - generates initial value for unions             */
+/*-----------------------------------------------------------------*/
+void pic16_printIvalUnion (symbol * sym, sym_link * type,
+                 initList * ilist, char ptype, void *p)
+{
+  //symbol *sflds;
+  initList *iloop = NULL;
+  int i, size;
+
+
+#if DEBUG_PRINTIVAL
+  fprintf(stderr, "%s\n",__FUNCTION__);
+#endif
+
+  iloop = ilist;
+  i = 0;
+  while (iloop)
+  {
+    i++;
+    iloop = iloop->next;
+  } // while
+
+  size = -1;
+  if (type) size = SPEC_STRUCT(type)->size;
+
+  if (i == 1 && size >= 0 && size <= sizeof(long))
+  {
+    unsigned long val = ulFromVal (list2val(ilist));
+    while (size--)
+    {
+      pic16_emitDB(val, ptype, p);
+      val >>= 8;
+    } // while
+    return;
+  } // if
+
+  fprintf( stderr, "INCOMPLETE SUPPORT FOR INITIALIZED union---FALLING BACK TO struct\n" );
+  fprintf( stderr, "This is a bug. Please file a bug-report with your source attached.\n" );
+  pic16_printIvalStruct( sym, type, ilist, ptype, p );
+}
+
+static int
+pic16_isUnion( symbol *sym, sym_link *type )
+{
+  if (type && SPEC_STRUCT(type)->type == UNION) return 1;
+  return 0;
+}
+
 /*--------------------------------------------------------------------------*/
 /* pic16_printIvalCharPtr - generates initial values for character pointers */
 /*--------------------------------------------------------------------------*/
@@ -843,7 +936,10 @@ int pic16_printIvalCharPtr (symbol * sym, sym_link * type, value * val, char pty
      VR - Attempting to port this function to pic16 port - 8-Jun-2004
    */
 
-//     fprintf(stderr, "%s\n",__FUNCTION__);
+
+#if DEBUG_PRINTIVAL
+  fprintf(stderr, "%s\n",__FUNCTION__);
+#endif
 
   size = getSize (type);
 
@@ -851,7 +947,7 @@ int pic16_printIvalCharPtr (symbol * sym, sym_link * type, value * val, char pty
     {
       if (size == 1)            /* This appears to be Z80 specific?? */
         {
-         pic16_emitDS(val->name, ptype, p);
+          pic16_emitDS(val->name, ptype, p);
         }
       else if (size == 2)
         {
@@ -869,13 +965,13 @@ int pic16_printIvalCharPtr (symbol * sym, sym_link * type, value * val, char pty
             // this is a literal string
             type=CPOINTER;
           }
-          pic16_printGPointerType(val->name, sym->name, type, type, ptype, p);
+          pic16_printGPointerType(val->name, type, ptype, p);
         }
       else
         {
           fprintf (stderr, "*** internal error: unknown size in "
                    "printIvalCharPtr.\n");
-         assert(0);
+          assert(0);
         }
     }
   else
@@ -884,16 +980,16 @@ int pic16_printIvalCharPtr (symbol * sym, sym_link * type, value * val, char pty
       switch (size)
         {
         case 1:
-         pic16_emitDB(pic16aopLiteral(val, 0), ptype, p);
+          pic16_emitDB(pic16aopLiteral(val, 0), ptype, p);
           break;
         case 2:
-         pic16_emitDB(pic16aopLiteral(val, 0), ptype, p);
-         pic16_emitDB(pic16aopLiteral(val, 1), ptype, p);
+          pic16_emitDB(pic16aopLiteral(val, 0), ptype, p);
+          pic16_emitDB(pic16aopLiteral(val, 1), ptype, p);
           break;
         case 3:
-         pic16_emitDB(pic16aopLiteral(val, 0), ptype, p);
-         pic16_emitDB(pic16aopLiteral(val, 1), ptype, p);
-         pic16_emitDB(pic16aopLiteral(val, 2), ptype, p);
+          pic16_emitDB(pic16aopLiteral(val, 0), ptype, p);
+          pic16_emitDB(pic16aopLiteral(val, 1), ptype, p);
+          pic16_emitDB(pic16aopLiteral(val, 2), ptype, p);
           break;
 
         default:
@@ -901,9 +997,9 @@ int pic16_printIvalCharPtr (symbol * sym, sym_link * type, value * val, char pty
         }
     }
 
-  if (val->sym && val->sym->isstrlit) {        // && !isinSet(statsg->syms, val->sym)) {
-       if(ptype == 'p' && !isinSet(statsg->syms, val->sym))addSet (&statsg->syms, val->sym);
-       else if(ptype == 'f' /*&& !isinSet(rel_idataSymSet, val->sym)*/)addSet(&rel_idataSymSet, val->sym);
+  if (val->sym && val->sym->isstrlit) { // && !isinSet(statsg->syms, val->sym)) {
+        if(ptype == 'p' && !isinSet(statsg->syms, val->sym))addSet (&statsg->syms, val->sym);
+        else if(ptype == 'f' /*&& !isinSet(rel_idataSymSet, val->sym)*/)addSet(&rel_idataSymSet, val->sym);
   }
 
   return 1;
@@ -917,6 +1013,11 @@ void pic16_printIvalFuncPtr (sym_link * type, initList * ilist, char ptype, void
   value *val;
   int dLvl = 0;
 
+
+#if DEBUG_PRINTIVAL
+  fprintf(stderr, "%s\n",__FUNCTION__);
+#endif
+
   if (ilist)
     val = list2val (ilist);
   else
@@ -924,7 +1025,7 @@ void pic16_printIvalFuncPtr (sym_link * type, initList * ilist, char ptype, void
 
   if (!val) {
     // an error has been thrown already
-    val=constVal("0");
+    val = constCharVal (0);
   }
 
   if (IS_LITERAL(val->etype)) {
@@ -945,9 +1046,21 @@ void pic16_printIvalFuncPtr (sym_link * type, initList * ilist, char ptype, void
 
   /* now generate the name */
   if (!val->sym) {
-      pic16_printPointerType (val->name, ptype, p);
+      pic16_printGPointerType (val->name, DCL_TYPE(val->type), ptype, p);
   } else {
-      pic16_printPointerType (val->sym->rname, ptype, p);
+      pic16_printGPointerType (val->sym->rname, DCL_TYPE(val->type), ptype, p);
+
+      if(IS_FUNC(val->sym->type) && !val->sym->used && !IS_STATIC(val->sym->etype)) {
+
+        if(!checkSym(publics, val->sym))
+          if(checkAddSym(&externs, val->sym) && (ptype == 'f')) {
+            /* this has not been declared as extern
+             * so declare it as a 'late extern' just after the symbol */
+            fprintf((FILE *)p, ";\tdeclare symbol as extern\n");
+            fprintf((FILE *)p, "\textern\t%s\n", val->sym->rname);
+            fprintf((FILE *)p, ";\tcontinue variable declaration\n");
+          }
+      }
   }
 
   return;
@@ -963,8 +1076,8 @@ void pic16_printIvalPtr (symbol * sym, sym_link * type, initList * ilist, char p
   int size;
 
 #if 0
-       fprintf(stderr, "%s:%d initialising pointer: %s size: %d\n", __FILE__, __LINE__,
-               sym->rname, getSize(sym->type));
+        fprintf(stderr, "%s:%d initialising pointer: %s size: %d\n", __FILE__, __LINE__,
+                sym->rname, getSize(sym->type));
 #endif
 
   /* if deep then   */
@@ -998,7 +1111,7 @@ void pic16_printIvalPtr (symbol * sym, sym_link * type, initList * ilist, char p
       switch (getSize (type))
         {
         case 1:
-            pic16_emitDB((unsigned int)floatFromVal(val) & 0xff, ptype, p);
+            pic16_emitDB((unsigned int) ulFromVal (val) & 0xff, ptype, p);
             break;
         case 2:
             pic16_emitDB(pic16aopLiteral(val, 0), ptype, p);
@@ -1008,8 +1121,10 @@ void pic16_printIvalPtr (symbol * sym, sym_link * type, initList * ilist, char p
             pic16_emitDB(pic16aopLiteral(val, 0), ptype, p);
             pic16_emitDB(pic16aopLiteral(val, 1), ptype, p);
             pic16_emitDB(pic16aopLiteral(val, 2), ptype, p);
+            break;
         default:
-               assert(0);
+                fprintf(stderr, "%s:%d size = %d\n", __FILE__, __LINE__, getSize(type));
+                assert(0);
         }
       return;
     }
@@ -1027,11 +1142,10 @@ void pic16_printIvalPtr (symbol * sym, sym_link * type, initList * ilist, char p
     }
   else if (size == 3)
     {
-      pic16_printGPointerType (val->name, sym->name, (IS_PTR(type)?DCL_TYPE(type):PTR_TYPE(SPEC_OCLS(sym->etype))),
-                         (IS_PTR (val->type) ? DCL_TYPE (val->type) :
-                          PTR_TYPE (SPEC_OCLS (val->etype))), ptype, p);
+      pic16_printGPointerType (val->name, (IS_PTR(type)?DCL_TYPE(type):PTR_TYPE(SPEC_OCLS(sym->etype))),
+                          ptype, p);
     } else
-       assert(0);
+        assert(0);
   return;
 }
 
@@ -1043,28 +1157,34 @@ void pic16_printIvalPtr (symbol * sym, sym_link * type, initList * ilist, char p
 void pic16_printIval (symbol * sym, sym_link * type, initList * ilist, char ptype, void *p)
 {
 //  sym_link *itype;
-  
+
   if (!p)
     return;
 
 #if 0
-       fprintf(stderr, "%s:%d generating init for %s\n", __FILE__, __LINE__, sym->name);
-       fprintf(stderr, "%s: IS_STRUCT: %d  IS_ARRAY: %d  IS_PTR: %d  IS_SPEC: %d\n", sym->name,
-               IS_STRUCT(type), IS_ARRAY(type), IS_PTR(type), IS_SPEC(type));
+        fprintf(stderr, "%s:%d generating init for %s\n", __FILE__, __LINE__, sym->name);
+        fprintf(stderr, "%s: IS_STRUCT: %d  IS_ARRAY: %d  IS_PTR: %d  IS_SPEC: %d\n", sym->name,
+                IS_STRUCT(type), IS_ARRAY(type), IS_PTR(type), IS_SPEC(type));
 #endif
-       
+
   /* if structure then */
   if (IS_STRUCT (type))
     {
-//      fprintf(stderr,"%s struct\n",__FUNCTION__);
-      pic16_printIvalStruct (sym, type, ilist, ptype, p);
+      if (pic16_isUnion(sym, type))
+        {
+          //fprintf(stderr,"%s union\n",__FUNCTION__);
+          pic16_printIvalUnion (sym, type, ilist, ptype, p);
+        } else {
+          //fprintf(stderr,"%s struct\n",__FUNCTION__);
+          pic16_printIvalStruct (sym, type, ilist, ptype, p);
+        }
       return;
     }
 
   /* if this is an array */
   if (IS_ARRAY (type))
     {
-//     fprintf(stderr,"%s array\n",__FUNCTION__);
+//      fprintf(stderr,"%s array\n",__FUNCTION__);
       pic16_printIvalArray (sym, type, ilist, ptype, p);
       return;
     }
@@ -1114,7 +1234,7 @@ void pic16_printIval (symbol * sym, sym_link * type, initList * ilist, char ptyp
   /* if type is SPECIFIER */
   if (IS_SPEC (type))
     {
-//     fprintf(stderr,"%s spec\n",__FUNCTION__);
+//      fprintf(stderr,"%s spec\n",__FUNCTION__);
       pic16_printIvalType (sym, type, ilist, ptype, p);
       return;
     }
@@ -1144,6 +1264,7 @@ static void
 pic16emitStaticSeg (memmap * map)
 {
   symbol *sym;
+  static int didcode=0;
 
   //fprintf(stderr, "%s\n",__FUNCTION__);
 
@@ -1155,146 +1276,154 @@ pic16emitStaticSeg (memmap * map)
     {
 
 #if 0
-       fprintf(stderr, "%s\t%s: sym: %s\tused: %d\tSPEC_ABSA: %d\tSPEC_AGGREGATE: %d\tCODE: %d\n\
+        fprintf(stderr, "%s\t%s: sym: %s\tused: %d\tSPEC_ABSA: %d\tSPEC_AGGREGATE: %d\tCODE: %d\n\
 CODESPACE: %d\tCONST: %d\tPTRCONST: %d\tSPEC_CONST: %d\n", __FUNCTION__,
-               map->sname, sym->name, sym->used, SPEC_ABSA(sym->etype), IS_AGGREGATE(sym->type),
-               IS_CODE(sym->etype), IN_CODESPACE( SPEC_OCLS(sym->etype)), IS_CONSTANT(sym->etype),
-               IS_PTR_CONST(sym->etype), SPEC_CONST(sym->etype));
-       printTypeChain( sym->type, stderr );
-       fprintf(stderr, "\n");
+                map->sname, sym->name, sym->used, SPEC_ABSA(sym->etype), IS_AGGREGATE(sym->type),
+                IS_CODE(sym->etype), IN_CODESPACE( SPEC_OCLS(sym->etype)), IS_CONSTANT(sym->etype),
+                IS_PTR_CONST(sym->etype), SPEC_CONST(sym->etype));
+        printTypeChain( sym->type, stderr );
+        fprintf(stderr, "\n");
 #endif
 
         if(SPEC_ABSA(sym->etype) && PIC16_IS_CONFIG_ADDRESS(SPEC_ADDR(sym->etype))) {
-               pic16_assignConfigWordValue(SPEC_ADDR(sym->etype),
-                       (int) floatFromVal(list2val(sym->ival)));
+                pic16_assignConfigWordValue(SPEC_ADDR(sym->etype),
+                        (int) ulFromVal (list2val(sym->ival)));
 
-               continue;
+                continue;
         }
 
         if(SPEC_ABSA(sym->etype) && PIC16_IS_IDLOC_ADDRESS(SPEC_ADDR(sym->etype))) {
-               pic16_assignIdByteValue(SPEC_ADDR(sym->etype),
-                       (char) floatFromVal(list2val(sym->ival)));
+                pic16_assignIdByteValue(SPEC_ADDR(sym->etype),
+                        (char) ulFromVal (list2val(sym->ival)));
 
-               continue;
+                continue;
         }
 
-       /* if it is "extern" then do nothing */
-       if (IS_EXTERN (sym->etype) && !SPEC_ABSA(sym->etype)) {
-               checkAddSym(&externs, sym);
-         continue;
-       }
+        /* if it is "extern" then do nothing */
+        if (IS_EXTERN (sym->etype)/* && !SPEC_ABSA(sym->etype)*/) {
+                checkAddSym(&externs, sym);
+          continue;
+        }
 
-       /* if it is not static add it to the public
-          table */
-       if (!IS_STATIC (sym->etype)) {
-               /* do not emit if it is a config word declaration */
-               checkAddSym(&publics, sym);
-       }
+        /* if it is not static add it to the public
+           table */
+        if (!IS_STATIC (sym->etype)) {
+                /* do not emit if it is a config word declaration */
+                checkAddSym(&publics, sym);
+        }
 
       /* print extra debug info if required */
       if (options.debug || sym->level == 0) {
-         /* NOTE to me - cdbFile may be null in which case,
-          * the sym name will be printed to stdout. oh well */
-          debugFile->writeSymbol(sym);
+          /* NOTE to me - cdbFile may be null in which case,
+           * the sym name will be printed to stdout. oh well */
+           debugFile->writeSymbol(sym);
       }
-        
+
       /* if it has an absolute address */
       if (SPEC_ABSA (sym->etype)) {
-//             fprintf(stderr, "%s:%d spec_absa is true for symbol: %s\n",
-//                     __FILE__, __LINE__, sym->name);
-                       
-         /* if it has an initial value */
-         if (sym->ival)
-           {
-             pBlock *pb;
+//              fprintf(stderr, "%s:%d spec_absa is true for symbol: %s\n",
+//                      __FILE__, __LINE__, sym->name);
+
+          /* if it has an initial value */
+          if (sym->ival)
+            {
+              pBlock *pb;
               symbol *asym;
               absSym *abSym;
               pCode *pcf;
-              
+
               /* symbol has absolute address and initial value */
-             noAlloc++;
-             resolveIvalSym (sym->ival, sym->type);
-             asym = newSymbol(sym->rname, 0);
-             abSym = Safe_calloc(1, sizeof(absSym));
-             abSym->name = Safe_strdup( sym->rname );
-             abSym->address = SPEC_ADDR( sym->etype );
-             addSet(&absSymSet, abSym);
-             
-             pb = pic16_newpCodeChain(NULL, 'A',pic16_newpCodeCharP("; Starting pCode block for absolute Ival"));
-             pic16_addpBlock(pb);
-
-             pcf = pic16_newpCodeFunction(moduleName, asym->name);
-             PCF(pcf)->absblock = 1;
-             
-             pic16_addpCode2pBlock(pb,pcf);
-             pic16_addpCode2pBlock(pb,pic16_newpCodeLabel(sym->rname,-1));
-//           fprintf(stderr, "%s:%d [1] generating init for label: %s\n", __FILE__, __LINE__, sym->rname);
-             pic16_printIval(sym, sym->type, sym->ival, 'p', (void *)pb);
+              noAlloc++;
+              resolveIvalSym (sym->ival, sym->type);
+              asym = newSymbol(sym->rname, 0);
+              abSym = Safe_calloc(1, sizeof(absSym));
+              strcpy(abSym->name, sym->rname);
+              abSym->address = SPEC_ADDR( sym->etype );
+              addSet(&absSymSet, abSym);
+
+              pb = pic16_newpCodeChain(NULL, 'A', pic16_newpCodeCharP("; Starting pCode block for absolute Ival"));
+              pic16_addpBlock(pb);
+
+              pcf = pic16_newpCodeFunction(moduleName, asym->name);
+              PCF(pcf)->absblock = 1;
+
+              pic16_addpCode2pBlock(pb,pcf);
+              pic16_addpCode2pBlock(pb,pic16_newpCodeLabel(sym->rname,-1));
+              //fprintf(stderr, "%s:%d [1] generating init for label: %s\n", __FILE__, __LINE__, sym->rname);
+              pic16_printIval(sym, sym->type, sym->ival, 'p', (void *)pb);
               pic16_flushDB('p', (void *)pb);
 
-             pic16_addpCode2pBlock(pb, pic16_newpCodeFunction(NULL, NULL));
-             noAlloc--;
-           }
-         else
-           {
-
-             /* symbol has absolute address but no initial value */
-             
-             /* allocate space */
-             fprintf (code->oFile, "%s:\n", sym->rname);
-
-             /* special case for character strings */
-             if (IS_ARRAY (sym->type) && IS_CHAR (sym->type->next) &&
-                 SPEC_CVAL (sym->etype).v_char) {
-
-//             fprintf(stderr, "%s:%d printing code string from %s\n", __FILE__, __LINE__, sym->rname);
-
-               pic16_pCodeConstString(sym->rname , SPEC_CVAL (sym->etype).v_char);
-             } else {
-               assert(0);
-             }
-           }
-  
-       } else {
-//             fprintf(stderr, "%s:%d spec_absa is false for symbol: %s\n",
-//                     __FILE__, __LINE__, sym->name);
-
-         /* if it has an initial value */
-         if (sym->ival) {
-             pBlock *pb;
-
-             /* symbol doesn't have absolute address but has initial value */
-             fprintf (code->oFile, "%s:\n", sym->rname);
-             noAlloc++;
-             resolveIvalSym (sym->ival, sym->type);
-
-             pb = pic16_newpCodeChain(NULL, 'P',pic16_newpCodeCharP("; Starting pCode block for Ival"));
-             pic16_addpBlock(pb);
-
-//           fprintf(stderr, "%s:%d [2] generating init for label: %s\n", __FILE__, __LINE__, sym->rname);
-
-             pic16_addpCode2pBlock(pb,pic16_newpCodeLabel(sym->rname,-1));
-             pic16_printIval(sym, sym->type, sym->ival, 'p', (void *)pb);
+              pic16_addpCode2pBlock(pb, pic16_newpCodeFunction(NULL, NULL));
+              noAlloc--;
+            }
+          else
+            {
+
+              /* symbol has absolute address but no initial value */
+
+              /* allocate space */
+              dbuf_printf (&code->oBuf, "%s:\n", sym->rname);
+
+              /* special case for character strings */
+              if (IS_ARRAY (sym->type) && IS_CHAR (sym->type->next) &&
+                  SPEC_CVAL (sym->etype).v_char) {
+
+//              fprintf(stderr, "%s:%d printing code string from %s\n", __FILE__, __LINE__, sym->rname);
+
+                pic16_pCodeConstString(sym->rname , SPEC_CVAL (sym->etype).v_char);
+              } else {
+                fprintf (stderr, "%s:%u(%s): do not know how to intialize symbol %s\n", __FILE__, __LINE__, __FUNCTION__, sym->rname);
+                assert(0);
+              }
+            }
+
+        } else {
+//              fprintf(stderr, "%s:%d spec_absa is false for symbol: %s\n",
+//                      __FILE__, __LINE__, sym->name);
+
+
+          /* if it has an initial value */
+          if (sym->ival) {
+              pBlock *pb;
+
+              /* symbol doesn't have absolute address but has initial value */
+              dbuf_printf (&code->oBuf, "%s:\n", sym->rname);
+              ++noAlloc;
+              resolveIvalSym (sym->ival, sym->type);
+
+              pb = pic16_newpCodeChain(NULL, 'P',pic16_newpCodeCharP("; Starting pCode block for Ival"));
+              pic16_addpBlock(pb);
+
+              if(!didcode) {
+                /* make sure that 'code' directive is emitted before, once */
+                pic16_addpCode2pBlock(pb, pic16_newpCodeAsmDir("code", NULL));
+
+                didcode++;
+              }
+
+              pic16_addpCode2pBlock(pb,pic16_newpCodeLabel(sym->rname,-1));
+              //fprintf(stderr, "%s:%d [2] generating init for label: %s\n", __FILE__, __LINE__, sym->rname);
+              pic16_printIval(sym, sym->type, sym->ival, 'p', (void *)pb);
               pic16_flushDB('p', (void *)pb);
-             noAlloc--;
-           } else {
-
-             /* symbol doesn't have absolute address and no initial value */
-             /* allocate space */
-//           fprintf(stderr, "%s:%d [3] generating init for label: %s\n", __FILE__, __LINE__, sym->rname);
-             fprintf (code->oFile, "%s:\n", sym->rname);
-             /* special case for character strings */
-             if (IS_ARRAY (sym->type) && IS_CHAR (sym->type->next) &&
-                 SPEC_CVAL (sym->etype).v_char) {
-               
-//             fprintf(stderr, "%s:%d printing code string from %s\n", __FILE__, __LINE__, sym->rname);
-
-               pic16_pCodeConstString(sym->rname , SPEC_CVAL (sym->etype).v_char);
-             } else {
-               assert(0);
-             }
-           }
-       }
+              --noAlloc;
+            } else {
+
+              /* symbol doesn't have absolute address and no initial value */
+              /* allocate space */
+//            fprintf(stderr, "%s:%d [3] generating init for label: %s\n", __FILE__, __LINE__, sym->rname);
+              dbuf_printf (&code->oBuf, "%s:\n", sym->rname);
+              /* special case for character strings */
+              if (IS_ARRAY (sym->type) && IS_CHAR (sym->type->next) &&
+                  SPEC_CVAL (sym->etype).v_char) {
+
+//              fprintf(stderr, "%s:%d printing code string for %s\n", __FILE__, __LINE__, sym->rname);
+
+                pic16_pCodeConstString(sym->rname , SPEC_CVAL (sym->etype).v_char);
+              } else {
+                assert(0);
+              }
+            }
+        }
     }
 
 }
@@ -1307,22 +1436,22 @@ void pic16_emitConfigRegs(FILE *of)
 {
   int i;
 
-       for(i=0;i<pic16->cwInfo.confAddrEnd-pic16->cwInfo.confAddrStart;i++)
-               if(pic16->cwInfo.crInfo[i].emit)        //mask != -1)
-                       fprintf (of, "\t__config 0x%x, 0x%hhx\n",
-                               pic16->cwInfo.confAddrStart+i,
-                               pic16->cwInfo.crInfo[i].value);
+        for(i=0;i<=(pic16->cwInfo.confAddrEnd-pic16->cwInfo.confAddrStart);i++)
+                if(pic16->cwInfo.crInfo[i].emit)        //mask != -1)
+                        fprintf (of, "\t__config 0x%x, 0x%hhx\n",
+                                pic16->cwInfo.confAddrStart+i,
+                                pic16->cwInfo.crInfo[i].value);
 }
 
 void pic16_emitIDRegs(FILE *of)
 {
   int i;
 
-       for(i=0;i<pic16->idInfo.idAddrEnd-pic16->idInfo.idAddrStart;i++)
-               if(pic16->idInfo.irInfo[i].emit)
-                       fprintf (of, "\t__idlocs 0x%06x, 0x%hhx\n",
-                               pic16->idInfo.idAddrStart+i,
-                               pic16->idInfo.irInfo[i].value);
+        for(i=0;i<=(pic16->idInfo.idAddrEnd-pic16->idInfo.idAddrStart);i++)
+                if(pic16->idInfo.irInfo[i].emit)
+                        fprintf (of, "\t__idlocs 0x%06x, 0x%hhx\n",
+                                pic16->idInfo.idAddrStart+i,
+                                pic16->idInfo.irInfo[i].value);
 }
 
 
@@ -1339,35 +1468,37 @@ pic16emitMaps ()
   pic16emitRegularMap (sfrbit, FALSE, FALSE);
   pic16emitRegularMap (code, TRUE, FALSE);
   pic16emitStaticSeg (statsg);
+  pic16emitStaticSeg (c_abs);
 }
 
 /*-----------------------------------------------------------------*/
 /* createInterruptVect - creates the interrupt vector              */
 /*-----------------------------------------------------------------*/
 static void
-pic16createInterruptVect (FILE * vFile)
+pic16createInterruptVect (struct dbuf_s * vBuf)
 {
-       /* if the main is only a prototype ie. no body then do nothing */
+        /* if the main is only a prototype ie. no body then do nothing */
 #if 0
-       if (!IFFUNC_HASBODY(mainf->type)) {
-               /* if ! compile only then main function should be present */
-               if (!options.cc_only)
-                       werror (E_NO_MAIN);
-               return;
-       }
+        if (!IFFUNC_HASBODY(mainf->type)) {
+                /* if ! compile only then main function should be present */
+                if (!options.cc_only)
+                        werror (E_NO_MAIN);
+                return;
+        }
+#endif
+#if 0
+        if((!pic16_options.omit_ivt) || (pic16_options.omit_ivt && pic16_options.leave_reset)) {
+                dbuf_printf (vBuf, ";\t.area\t%s\n", CODE_NAME);
+                dbuf_printf (vBuf, ".intvecs\tcode\t0x%06x\n", pic16_options.ivt_loc);
+
+                /* this is an overkill since WE are the port,
+                 * and we know if we have a genIVT function! */
+                if(port->genIVT) {
+                        port->genIVT(vFile, interrupts, maxInterrupts);
+                }
+        }
 #endif
 
-       if((!pic16_options.omit_ivt) || (pic16_options.omit_ivt && pic16_options.leave_reset)) {
-               fprintf (vFile, ";\t.area\t%s\n", CODE_NAME);
-               fprintf(vFile, ".intvecs\tcode\t0x%06x\n", pic16_options.ivt_loc);
-
-               /* this is an overkill since WE are the port,
-                * and we know if we have a genIVT function! */
-               if(port->genIVT) {
-                       port->genIVT(vFile, interrupts, maxInterrupts);
-               }
-       }
-       
 }
 
 
@@ -1377,10 +1508,54 @@ pic16createInterruptVect (FILE * vFile)
 static void
 pic16initialComments (FILE * afile)
 {
-  initialComments (afile);
-  fprintf (afile, "; PIC16 port for the Microchip 16-bit core micros\n");
-  fprintf (afile, iComments2);
+        initialComments (afile);
+        fprintf (afile, "; PIC16 port for the Microchip 16-bit core micros\n");
+        if(xinst)
+          fprintf (afile, "; * Extended Instruction Set\n");
+
+        if(pic16_mplab_comp)
+                fprintf(afile, "; * MPLAB/MPASM/MPASMWIN/MPLINK compatibility mode enabled\n");
+        fprintf (afile, iComments2);
+
+        if(options.debug) {
+                fprintf (afile, "\n\t.ident \"SDCC version %s #%s [pic16 port]%s\"\n",
+                                SDCC_VERSION_STR, getBuildNumber(), (!xinst?"":" {extended}") );
+        }
+}
+
+int
+pic16_stringInSet(const char *str, set **world, int autoAdd)
+{
+  char *s;
 
+  if (!str) return 1;
+  assert(world);
+
+  for (s = setFirstItem(*world); s; s = setNextItem(*world))
+  {
+    /* found in set */
+    if (0 == strcmp(s, str)) return 1;
+  }
+
+  /* not found */
+  if (autoAdd) addSet(world, Safe_strdup(str));
+  return 0;
+}
+
+static int
+pic16_emitSymbolIfNew(FILE *file, const char *fmt, const char *sym, int checkLocals)
+{
+  static set *emitted = NULL;
+
+  if (!pic16_stringInSet(sym, &emitted, 1)) {
+    /* sym was not in emittedSymbols */
+    if (!checkLocals || !pic16_stringInSet(sym, &pic16_localFunctions, 0)) {
+      /* sym is not a locally defined function---avoid bug #1443651 */
+      fprintf( file, fmt, sym );
+      return 0;
+    }
+  }
+  return 1;
 }
 
 /*-----------------------------------------------------------------*/
@@ -1391,12 +1566,14 @@ pic16printPublics (FILE *afile)
 {
   symbol *sym;
 
-       fprintf (afile, "%s", iComments2);
-       fprintf (afile, "; public variables in this module\n");
-       fprintf (afile, "%s", iComments2);
+        fprintf (afile, "\n%s", iComments2);
+        fprintf (afile, "; public variables in this module\n");
+        fprintf (afile, "%s", iComments2);
 
-       for(sym = setFirstItem (publics); sym; sym = setNextItem (publics))
-               fprintf(afile, "\tglobal %s\n", sym->rname);
+        for(sym = setFirstItem (publics); sym; sym = setNextItem (publics))
+          /* sanity check */
+          if(!IS_STATIC(sym->etype))
+                pic16_emitSymbolIfNew(afile, "\tglobal %s\n", sym->rname, 0);
 }
 
 /*-----------------------------------------------------------------*/
@@ -1407,27 +1584,31 @@ pic16_printExterns(FILE *afile)
 {
   symbol *sym;
 
-       fprintf(afile, "%s", iComments2);
-       fprintf(afile, "; extern variables in this module\n");
-       fprintf(afile, "%s", iComments2);
-       
-       for(sym = setFirstItem(externs); sym; sym = setNextItem(externs))
-               fprintf(afile, "\textern %s\n", sym->rname);
+        /* print nothing if no externs to declare */
+        if(!elementsInSet(externs) && !elementsInSet(pic16_builtin_functions))
+                return;
+
+        fprintf(afile, "\n%s", iComments2);
+        fprintf(afile, "; extern variables in this module\n");
+        fprintf(afile, "%s", iComments2);
+
+        for(sym = setFirstItem(externs); sym; sym = setNextItem(externs))
+                pic16_emitSymbolIfNew(afile, "\textern %s\n", sym->rname, 1);
 
-       for(sym = setFirstItem(pic16_builtin_functions); sym; sym = setNextItem(pic16_builtin_functions))
-               fprintf(afile, "\textern _%s\n", sym->name);
+        for(sym = setFirstItem(pic16_builtin_functions); sym; sym = setNextItem(pic16_builtin_functions))
+                pic16_emitSymbolIfNew(afile, "\textern _%s\n", sym->name, 1);
 }
 
 /*-----------------------------------------------------------------*/
 /* emitOverlay - will emit code for the overlay stuff              */
 /*-----------------------------------------------------------------*/
 static void
-pic16emitOverlay (FILE * afile)
+pic16emitOverlay (struct dbuf_s *aBuf)
 {
   set *ovrset;
 
   if (!elementsInSet (ovrSetSets))
-    fprintf (afile, ";\t.area\t%s\n", port->mem.overlay_name);
+    dbuf_printf (aBuf, ";\t.area\t%s\n", port->mem.overlay_name);
 
   /* for each of the sets in the overlay segment do */
   for (ovrset = setFirstItem (ovrSetSets); ovrset;
@@ -1437,94 +1618,96 @@ pic16emitOverlay (FILE * afile)
       symbol *sym;
 
       if (elementsInSet (ovrset))
-       {
-         /* this dummy area is used to fool the assembler
-            otherwise the assembler will append each of these
-            declarations into one chunk and will not overlay
-            sad but true */
-         fprintf (afile, ";\t.area _DUMMY\n");
-         /* output the area informtion */
-         fprintf (afile, ";\t.area\t%s\n", port->mem.overlay_name);    /* MOF */
-       }
+        {
+          /* this dummy area is used to fool the assembler
+             otherwise the assembler will append each of these
+             declarations into one chunk and will not overlay
+             sad but true */
+          dbuf_printf (aBuf, ";\t.area _DUMMY\n");
+          /* output the area informtion */
+          dbuf_printf (aBuf, ";\t.area\t%s\n", port->mem.overlay_name); /* MOF */
+        }
 
       for (sym = setFirstItem (ovrset); sym;
-          sym = setNextItem (ovrset))
-       {
-
-         /* if extern then do nothing */
-         if (IS_EXTERN (sym->etype))
-           continue;
-
-         /* if allocation required check is needed
-            then check if the symbol really requires
-            allocation only for local variables */
-         if (!IS_AGGREGATE (sym->type) &&
-             !(sym->_isparm && !IS_REGPARM (sym->etype))
-             && !sym->allocreq && sym->level)
-           continue;
-
-         /* if global variable & not static or extern
-            and addPublics allowed then add it to the public set */
-         if ((sym->_isparm && !IS_REGPARM (sym->etype))
-             && !IS_STATIC (sym->etype)) {
-//           fprintf(stderr, "%s:%d %s accessed\n", __FILE__, __LINE__, __FUNCTION__);
-             checkAddSym(&publics, sym);
-//         addSetHead (&publics, sym);
-         }
-
-         /* if extern then do nothing or is a function
-            then do nothing */
-         if (IS_FUNC (sym->type))
-           continue;
+           sym = setNextItem (ovrset))
+        {
 
-#if 0
-         /* print extra debug info if required */
-         if (options.debug || sym->level == 0)
-           {
-
-             cdbSymbol (sym, cdbFile, FALSE, FALSE);
-
-             if (!sym->level)
-               {               /* global */
-                 if (IS_STATIC (sym->etype))
-                   fprintf (afile, "F%s_", moduleName);        /* scope is file */
-                 else
-                   fprintf (afile, "G_");      /* scope is global */
-               }
-             else
-               /* symbol is local */
-               fprintf (afile, "L%s_",
-                        (sym->localof ? sym->localof->name : "-null-"));
-             fprintf (afile, "%s_%d_%d", sym->name, sym->level, sym->block);
-           }
-#endif
+          /* if extern then do nothing */
+          if (IS_EXTERN (sym->etype))
+            continue;
+
+          /* if allocation required check is needed
+             then check if the symbol really requires
+             allocation only for local variables */
+          if (!IS_AGGREGATE (sym->type) &&
+              !(sym->_isparm && !IS_REGPARM (sym->etype))
+              && !sym->allocreq && sym->level)
+            continue;
+
+          /* if global variable & not static or extern
+             and addPublics allowed then add it to the public set */
+          if ((sym->_isparm && !IS_REGPARM (sym->etype))
+              && !IS_STATIC (sym->etype)) {
+//            fprintf(stderr, "%s:%d %s accessed\n", __FILE__, __LINE__, __FUNCTION__);
+              checkAddSym(&publics, sym);
+//          addSetHead (&publics, sym);
+          }
+
+          /* if extern then do nothing or is a function
+             then do nothing */
+          if (IS_FUNC (sym->type))
+            continue;
+
+
+          /* if is has an absolute address then generate
+             an equate for this no need to allocate space */
+          if (SPEC_ABSA (sym->etype))
+            {
+
+              if (options.debug || sym->level == 0)
+                dbuf_printf (aBuf, " == 0x%04x\n", SPEC_ADDR (sym->etype));
 
-         /* if is has an absolute address then generate
-            an equate for this no need to allocate space */
-         if (SPEC_ABSA (sym->etype))
-           {
-
-             if (options.debug || sym->level == 0)
-               fprintf (afile, " == 0x%04x\n", SPEC_ADDR (sym->etype));
-
-             fprintf (afile, "%s\t=\t0x%04x\n",
-                      sym->rname,
-                      SPEC_ADDR (sym->etype));
-           }
-         else
-           {
-             if (options.debug || sym->level == 0)
-               fprintf (afile, "==.\n");
-
-             /* allocate space */
-             fprintf (afile, "%s:\n", sym->rname);
-             fprintf (afile, "\t.ds\t0x%04x\n", (unsigned int) getSize (sym->type) & 0xffff);
-           }
-
-       }
+              dbuf_printf (aBuf, "%s\t=\t0x%04x\n",
+                       sym->rname,
+                       SPEC_ADDR (sym->etype));
+            }
+          else
+            {
+              if (options.debug || sym->level == 0)
+                dbuf_printf (aBuf, "==.\n");
+
+              /* allocate space */
+              dbuf_printf (aBuf, "%s:\n", sym->rname);
+              dbuf_printf (aBuf, "\t.ds\t0x%04x\n", (unsigned int) getSize (sym->type) & 0xffff);
+            }
+
+        }
     }
 }
 
+void emitStatistics(FILE *asmFile)
+{
+  unsigned long isize, udsize, ramsize;
+  statistics.isize = pic16_countInstructions();
+  isize = (statistics.isize >= 0) ? statistics.isize : 0;
+  udsize = (statistics.udsize >= 0) ? statistics.udsize : 0;
+  ramsize = pic16 ? pic16->RAMsize : 0x200;
+  ramsize -= 256; /* ignore access bank and SFRs */
+  if (ramsize == 0) ramsize = 64; /* prevent division by zero (below) */
+
+  fprintf (asmFile, "\n\n; Statistics:\n");
+  fprintf (asmFile, "; code size:\t%5ld (0x%04lx) bytes (%5.2f%%)\n;           \t%5ld (0x%04lx) words\n",
+    isize, isize, (isize*100.0)/(128UL << 10),
+    isize>>1, isize>>1);
+  fprintf (asmFile, "; udata size:\t%5ld (0x%04lx) bytes (%5.2f%%)\n",
+    udsize, udsize, (udsize*100.0) / (1.0 * ramsize));
+  fprintf (asmFile, "; access size:\t%5ld (0x%04lx) bytes\n",
+    statistics.intsize, statistics.intsize);
+
+  fprintf (asmFile, "\n\n");
+}
+
+
 
 /*-----------------------------------------------------------------*/
 /* glue - the final glue that hold the whole thing together        */
@@ -1532,248 +1715,229 @@ pic16emitOverlay (FILE * afile)
 void
 pic16glue ()
 {
-
-  FILE *vFile;
   FILE *asmFile;
-  FILE *ovrFile = tempfile();
+  struct dbuf_s ovrBuf;
+  struct dbuf_s vBuf;
 
+    dbuf_init(&ovrBuf, 4096);
+    dbuf_init(&vBuf, 4096);
 
-       mainf = newSymbol ("main", 0);
-       mainf->block = 0;
+    mainf = newSymbol ("main", 0);
+    mainf->block = 0;
 
-       mainf = findSymWithLevel(SymbolTab, mainf);
-#if 0
-       /* only if the main function exists */
-       if (!(mainf = findSymWithLevel (SymbolTab, mainf))) {
-               if (!options.cc_only)
-                       werror (E_NO_MAIN);
-               return;
-       }
-#endif
+    mainf = findSymWithLevel(SymbolTab, mainf);
+
+    pic16_pCodeInitRegisters();
+
+    if(pic16_options.no_crt && mainf && IFFUNC_HASBODY(mainf->type)) {
+      pBlock *pb = pic16_newpCodeChain(NULL,'X',pic16_newpCodeCharP("; Starting pCode block"));
+
+        pic16_addpBlock(pb);
 
-//     fprintf(stderr, "main function= %p (%s)\thas body= %d\n", mainf, (mainf?mainf->name:NULL), mainf?IFFUNC_HASBODY(mainf->type):-1);
-
-       addSetHead(&tmpfileSet,ovrFile);
-       pic16_pCodeInitRegisters();
-
-       if (mainf && IFFUNC_HASBODY(mainf->type)) {
-         pBlock *pb = pic16_newpCodeChain(NULL,'X',pic16_newpCodeCharP("; Starting pCode block"));
-
-               pic16_addpBlock(pb);
-
-               /* entry point @ start of CSEG */
-               pic16_addpCode2pBlock(pb,pic16_newpCodeLabel("__sdcc_program_startup",-1));
-
-               if(initsfpnt) {
-                       pic16_addpCode2pBlock(pb, pic16_newpCode(POC_LFSR,
-                               pic16_popGetLit2(1, pic16_newpCodeOpRegFromStr("_stack"))));
-                       pic16_addpCode2pBlock(pb, pic16_newpCode(POC_LFSR,
-                               pic16_popGetLit2(2, pic16_newpCodeOpRegFromStr("_stack"))));
-               }
-
-               /* put in the call to main */
-               pic16_addpCode2pBlock(pb,pic16_newpCode(POC_CALL,pic16_newpCodeOp("_main",PO_STR)));
-
-               if (options.mainreturn) {
-                       pic16_addpCode2pBlock(pb,pic16_newpCodeCharP(";\treturn from main will return to caller\n"));
-                       pic16_addpCode2pBlock(pb,pic16_newpCode(POC_RETURN,NULL));
-               } else {
-                       pic16_addpCode2pBlock(pb,pic16_newpCodeCharP(";\treturn from main will lock up\n"));
-                       pic16_addpCode2pBlock(pb,pic16_newpCode(POC_GOTO,pic16_newpCodeOp("$",PO_STR)));
-               }
-       }
-
-       /* At this point we've got all the code in the form of pCode structures */
-       /* Now it needs to be rearranged into the order it should be placed in the */
-       /* code space */
-
-       pic16_movepBlock2Head('P');              // Last
-       pic16_movepBlock2Head(code->dbName);
-       pic16_movepBlock2Head('X');
-       pic16_movepBlock2Head(statsg->dbName);   // First
-
-       /* print the global struct definitions */
-//     if (options.debug)
-//             cdbStructBlock (0);     //,cdbFile);
-
-       vFile = tempfile();
-       /* PENDING: this isnt the best place but it will do */
-       if (port->general.glue_up_main) {
-               /* create the interrupt vector table */
-               pic16createInterruptVect (vFile);
-       }
-
-       addSetHead(&tmpfileSet,vFile);
-    
-       /* emit code for the all the variables declared */
-       pic16emitMaps ();
-       /* do the overlay segments */
-       pic16emitOverlay(ovrFile);
-       pic16_AnalyzepCode('*');
+        /* entry point @ start of CSEG */
+        pic16_addpCode2pBlock(pb,pic16_newpCodeLabel("__sdcc_program_startup",-1));
+
+        if(initsfpnt) {
+          pic16_addpCode2pBlock(pb, pic16_newpCode(POC_LFSR,
+                  pic16_popGetLit2(1, pic16_newpCodeOpRegFromStr("_stack_end"))));
+          pic16_addpCode2pBlock(pb, pic16_newpCode(POC_LFSR,
+                  pic16_popGetLit2(2, pic16_newpCodeOpRegFromStr("_stack_end"))));
+        }
+
+        /* put in the call to main */
+        pic16_addpCode2pBlock(pb,pic16_newpCode(POC_CALL,pic16_newpCodeOp("_main",PO_STR)));
+
+        if (options.mainreturn) {
+          pic16_addpCode2pBlock(pb,pic16_newpCodeCharP(";\treturn from main will return to caller\n"));
+          pic16_addpCode2pBlock(pb,pic16_newpCode(POC_RETURN,NULL));
+        } else {
+          pic16_addpCode2pBlock(pb,pic16_newpCodeCharP(";\treturn from main will lock up\n"));
+          pic16_addpCode2pBlock(pb,pic16_newpCode(POC_GOTO,pic16_newpCodeOp("$",PO_STR)));
+        }
+    }
+
+    /* At this point we've got all the code in the form of pCode structures */
+    /* Now it needs to be rearranged into the order it should be placed in the */
+    /* code space */
+
+    pic16_movepBlock2Head('P');              // Last
+    pic16_movepBlock2Head(code->dbName);
+    pic16_movepBlock2Head('X');
+    pic16_movepBlock2Head(statsg->dbName);   // First
+
+    /* print the global struct definitions */
+
+    /* PENDING: this isnt the best place but it will do */
+    if (port->general.glue_up_main) {
+      /* create the interrupt vector table */
+      pic16createInterruptVect (&vBuf);
+    }
+
+    /* emit code for the all the variables declared */
+    pic16emitMaps ();
+
+    /* do the overlay segments */
+    pic16emitOverlay(&ovrBuf);
+    pic16_AnalyzepCode('*');
 
 #if 1
-       if(pic16_options.dumpcalltree) {
-         FILE *cFile;
-               sprintf(buffer, dstFileName);
-               strcat(buffer, ".calltree");
-               cFile = fopen(buffer, "w");
-               pic16_printCallTree( cFile );
-               fclose(cFile);
-       }
+    if(pic16_options.dumpcalltree) {
+      FILE *cFile;
+
+        sprintf(buffer, dstFileName);
+        strcat(buffer, ".calltree");
+        cFile = fopen(buffer, "w");
+        pic16_printCallTree( cFile );
+        fclose(cFile);
+    }
 #endif
 
-       pic16_InlinepCode();
-       pic16_AnalyzepCode('*');
-
-       if(pic16_debug_verbose)
-               pic16_pcode_test();
-
-       /* now put it all together into the assembler file */
-       /* create the assembler file name */
-       if ((noAssemble || options.c1mode)  && fullDstFileName) {
-               sprintf (buffer, fullDstFileName);
-       } else {
-               sprintf (buffer, dstFileName);
-               strcat (buffer, ".asm");
-       }
-
-       if (!(asmFile = fopen (buffer, "w"))) {
-               werror (E_FILE_OPEN_ERR, buffer);
-               exit (1);
-       }
-    
-       /* initial comments */
-       pic16initialComments (asmFile);
-
-       /* print module name */
-       fprintf(asmFile, "#FILE\t\"%s\"\n", fullSrcFileName);
-
-       /* Let the port generate any global directives, etc. */
-       if (port->genAssemblerPreamble) {
-               port->genAssemblerPreamble(asmFile);
-       }
-       
-       /* print the extern variables to this module */
-       pic16_printExterns(asmFile);
-       
-       /* print the global variables in this module */
-       pic16printPublics (asmFile);
+    pic16_InlinepCode();
+    pic16_AnalyzepCode('*');
 
-#if 0
-       /* copy the sfr segment */
-       fprintf (asmFile, "%s", iComments2);
-       fprintf (asmFile, "; special function registers\n");
-       fprintf (asmFile, "%s", iComments2);
-       copyFile (asmFile, sfr->oFile);
-#endif
 
-       /* Put all variables into a cblock */
-       pic16_AnalyzeBanking();
-       pic16_writeUsedRegs(asmFile);
+    if(pic16_debug_verbose)
+      pic16_pcode_test();
+
+    /* now put it all together into the assembler file */
+    /* create the assembler file name */
+    if((noAssemble || options.c1mode)  && fullDstFileName) {
+      sprintf (buffer, fullDstFileName);
+    } else {
+      sprintf (buffer, dstFileName);
+      strcat (buffer, ".asm");
+    }
+
+    if(!(asmFile = fopen (buffer, "w"))) {
+      werror (E_FILE_OPEN_ERR, buffer);
+      exit (1);
+    }
+
+    /* initial comments */
+    pic16initialComments (asmFile);
+
+    /* print module name */
+    if(options.debug)
+      fprintf(asmFile, "\t.file\t\"%s\"\n", fullSrcFileName);
+
+    /* Let the port generate any global directives, etc. */
+    if(port->genAssemblerPreamble) {
+      port->genAssemblerPreamble(asmFile);
+    }
+
+    /* Put all variables into a cblock */
+    pic16_AnalyzeBanking();
 
 #if 0
-       /* no xdata in pic */
-       /* if external stack then reserve space of it */
-       if (mainf && IFFUNC_HASBODY(mainf->type) && options.useXstack ) {
-               fprintf (asmFile, "%s", iComments2);
-               fprintf (asmFile, "; external stack \n");
-               fprintf (asmFile, "%s", iComments2);
-               fprintf (asmFile,";\t.area XSEG (XDATA)\n"); /* MOF */
-               fprintf (asmFile,";\t.ds 256\n");
-       }
+    if(pic16_options.opt_flags & OF_LR_SUPPORT) {
+      pic16_OptimizeLocalRegs();
+    }
 #endif
 
-#if 0  
-       /* no xdata in pic */
-       /* copy xtern ram data */
-       fprintf (asmFile, "%s", iComments2);
-       fprintf (asmFile, "; external ram data\n");
-       fprintf (asmFile, "%s", iComments2);
-       copyFile (asmFile, xdata->oFile);
+    /* remove redundant BANKSELs -- added by RN 2005-01-17 */
+    if(pic16_options.opt_banksel > 1) {
+      pic16_OptimizeBanksel();
+    }
+
+    /* turn GOTOs into BRAs -- added by RN 2004-11-16 */
+    if(pic16_options.opt_flags & OF_OPTIMIZE_GOTO) {
+      pic16_OptimizeJumps();
+    }
+
+    /* print the global variables in this module */
+    pic16printPublics (asmFile);
+
+    /* print the extern variables to this module */
+    pic16_printExterns(asmFile);
+
+    pic16_writeUsedRegs(asmFile);
+
+#if 0
+    /* no xdata in pic */
+    /* if external stack then reserve space of it */
+    if(mainf && IFFUNC_HASBODY(mainf->type) && options.useXstack ) {
+      fprintf (asmFile, "%s", iComments2);
+      fprintf (asmFile, "; external stack \n");
+      fprintf (asmFile, "%s", iComments2);
+      fprintf (asmFile,";\t.area XSEG (XDATA)\n"); /* MOF */
+      fprintf (asmFile,";\t.ds 256\n");
+    }
 #endif
 
 #if 0
-       /* copy the bit segment */
-       fprintf (asmFile, "%s", iComments2);
-       fprintf (asmFile, "; bit data\n");
-       fprintf (asmFile, "%s", iComments2);
-       copyFile (asmFile, bit->oFile);
+    /* no xdata in pic */
+    /* copy xtern ram data */
+    fprintf (asmFile, "%s", iComments2);
+    fprintf (asmFile, "; external ram data\n");
+    fprintf (asmFile, "%s", iComments2);
+    dbuf_write_and_destroy (&xdata->oBuf, asmFile);
 #endif
 
-       /* copy the interrupt vector table */
-       if(mainf && IFFUNC_HASBODY(mainf->type)) {
-               fprintf (asmFile, "%s", iComments2);
-               fprintf (asmFile, "; interrupt vector \n");
-               fprintf (asmFile, "%s", iComments2);
-               copyFile (asmFile, vFile);
-       }
-    
-       /* copy global & static initialisations */
-       fprintf (asmFile, "%s", iComments2);
-       fprintf (asmFile, "; global & static initialisations\n");
-       fprintf (asmFile, "%s", iComments2);
-    
 #if 0
-       /* copy over code */
-       fprintf (asmFile, "%s", iComments2);
-       fprintf (asmFile, "\tcode\n");
-       fprintf (asmFile, "%s", iComments2);
+    /* copy the bit segment */
+    fprintf (asmFile, "%s", iComments2);
+    fprintf (asmFile, "; bit data\n");
+    fprintf (asmFile, "%s", iComments2);
+    dbuf_write_and_destroy (&bit->oBuf, asmFile);
 #endif
 
-       if(pic16_debug_verbose)
-               fprintf(asmFile, "; A code from now on!\n");
-       pic16_copypCode(asmFile, 'A');
+    /* copy the interrupt vector table */
+    if(mainf && IFFUNC_HASBODY(mainf->type)) {
+      fprintf (asmFile, "\n%s", iComments2);
+      fprintf (asmFile, "; interrupt vector \n");
+      fprintf (asmFile, "%s", iComments2);
+      dbuf_write_and_destroy (&vBuf, asmFile);
+    }
 
+    /* copy global & static initialisations */
+    fprintf (asmFile, "\n%s", iComments2);
+    fprintf (asmFile, "; global & static initialisations\n");
+    fprintf (asmFile, "%s", iComments2);
 
-       if(mainf && IFFUNC_HASBODY(mainf->type)) {
-               fprintf(asmFile, "\tcode\n");
-               fprintf(asmFile,"__sdcc_gsinit_startup:\n");
+    if(pic16_debug_verbose)
+      fprintf(asmFile, "; A code from now on!\n");
 
-#if 0
-               /* FIXME 8051 legacy (?!) - VR 20-Jun-2003 */
-               /* if external stack is specified then the
-                * higher order byte of the xdatalocation is
-                * going into P2 and the lower order going into */
-       
-               if (options.useXstack) {
-                       fprintf(asmFile,";\tmov\tP2,#0x%02x\n",
-                                       (((unsigned int)options.xdata_loc) >> 8) & 0xff);
-                       fprintf(asmFile,";\tmov\t_spx,#0x%02x\n",
-                                       (unsigned int)options.xdata_loc & 0xff);
-               }
-#endif
-       }
+    pic16_copypCode(asmFile, 'A');
 
-//     copyFile (stderr, code->oFile);
+    if(pic16_options.no_crt) {
+      if(mainf && IFFUNC_HASBODY(mainf->type)) {
+        fprintf(asmFile, "\tcode\n");
+        fprintf(asmFile,"__sdcc_gsinit_startup:\n");
+      }
+    }
+
+//    dbuf_write_and_destroy (&code->oBuf, stderr);
 
-       fprintf(asmFile, "; I code from now on!\n");
-       pic16_copypCode(asmFile, 'I');
+    fprintf(asmFile, "; I code from now on!\n");
+    pic16_copypCode(asmFile, 'I');
 
-       if(pic16_debug_verbose)
-               fprintf(asmFile, "; dbName from now on!\n");
-       pic16_copypCode(asmFile, statsg->dbName);
+    if(pic16_debug_verbose)
+      fprintf(asmFile, "; dbName from now on!\n");
 
+    pic16_copypCode(asmFile, statsg->dbName);
+
+    if(pic16_options.no_crt) {
+      if (port->general.glue_up_main && mainf && IFFUNC_HASBODY(mainf->type)) {
+        fprintf (asmFile,"\tgoto\t__sdcc_program_startup\n");
+      }
+    }
 
-       if (port->general.glue_up_main && mainf && IFFUNC_HASBODY(mainf->type)) {
-               fprintf (asmFile,"\tgoto\t__sdcc_program_startup\n");
-       }
-       
+    if(pic16_debug_verbose)
+      fprintf(asmFile, "; X code from now on!\n");
 
-       if(pic16_debug_verbose)
-               fprintf(asmFile, "; X code from now on!\n");
-       pic16_copypCode(asmFile, 'X');
+    pic16_copypCode(asmFile, 'X');
 
-       if(pic16_debug_verbose)
-               fprintf(asmFile, "; M code from now on!\n");
-       pic16_copypCode(asmFile, 'M');
+    if(pic16_debug_verbose)
+      fprintf(asmFile, "; M code from now on!\n");
 
+    pic16_copypCode(asmFile, 'M');
 
-       pic16_copypCode(asmFile, code->dbName);
+    pic16_copypCode(asmFile, code->dbName);
 
-       pic16_copypCode(asmFile, 'P');
+    pic16_copypCode(asmFile, 'P');
 
-       fprintf (asmFile,"\tend\n");
-       fclose (asmFile);
+    emitStatistics(asmFile);
 
-       rm_tmpfiles();
+    fprintf (asmFile,"\tend\n");
+    fclose (asmFile);
 }