X-Git-Url: https://git.gag.com/?a=blobdiff_plain;f=src%2Fpic%2Fmain.c;h=7e87781be36730c5a78ca0f853c860c5530c0b98;hb=97bed346fed0a8e37ef2aa6da4f056be1293e828;hp=5c997639473249d599b91c5151222623d4ec4bc2;hpb=c766c5e82b158b1efc264c72453cd66dcebf8063;p=fw%2Fsdcc diff --git a/src/pic/main.c b/src/pic/main.c index 5c997639..7e87781b 100644 --- a/src/pic/main.c +++ b/src/pic/main.c @@ -9,19 +9,40 @@ #include "ralloc.h" #include "device.h" #include "SDCCutil.h" +#include "SDCCmacro.h" +#include "MySystem.h" +#include "glue.h" +#include "dbuf_string.h" +#include //#include "gen.h" +#define OPTION_STACK_SIZE "--stack-size" +#define OPTION_UDATA_SECTION_NAME "--udata-section-name" + static char _defaultRules[] = { #include "peeph.rul" }; -/* list of key words used by msc51 */ +pic14_options_t pic14_options; +extern int debug_verbose; /* from pcode.c */ +extern char *udata_section_name; + +static OPTION _pic14_poptions[] = + { + { 0, "--debug-xtra", &debug_verbose, "show more debug info in assembly output" }, + { 0, "--no-pcode-opt", &pic14_options.disable_df, "disable (slightly faulty) optimization on pCode" }, + { 0, OPTION_STACK_SIZE, &options.stack_size, "sets the size if the argument passing stack (default: 16, minimum: 4)", CLAT_INTEGER }, + { 0, OPTION_UDATA_SECTION_NAME, &udata_section_name, "set udata section name", CLAT_STRING }, + { 0, NULL, NULL, NULL } + }; + +/* list of key words used by pic14 */ static char *_pic14_keywords[] = { "at", - "bit", + //"bit", "code", "critical", "data", @@ -32,7 +53,7 @@ static char *_pic14_keywords[] = "pdata", "reentrant", "sfr", - "sbit", + //"sbit", "using", "xdata", "_data", @@ -47,168 +68,141 @@ static char *_pic14_keywords[] = void pCodeInitRegisters(void); -void pic14_assignRegisters (eBBlock ** ebbs, int count); +void pic14_assignRegisters (ebbIndex *); + +/* Also defined in gen.h, but the #include is commented out */ +/* for an unknowned reason. - EEP */ +void pic14_emitDebuggerSymbol (char *); -static int regParmFlg = 0; /* determine if we can register a parameter */ +static int regParmFlg = 0; /* determine if we can register a parameter */ static void _pic14_init (void) { asm_addTree (&asm_asxxxx_mapping); - pCodeInitRegisters(); + memset (&pic14_options, 0, sizeof (pic14_options)); } static void -_pic14_reset_regparm () +_pic14_reset_regparm (void) { regParmFlg = 0; } static int -_pic14_regparm (sym_link * l) +_pic14_regparm (sym_link * l, bool reentrant) { - /* for this processor it is simple - can pass only the first parameter in a register */ +/* for this processor it is simple + can pass only the first parameter in a register */ //if (regParmFlg) // return 0; - + regParmFlg++;// = 1; return 1; } -static int -_process_pragma(const char *sz) +static bool +_pic14_parseOptions (int *pargc, char **argv, int *i) { - static const char *WHITE = " \t"; - char *ptr = strtok((char *)sz, WHITE); - - if (startsWith (ptr, "memmap")) - { - char *start; - char *end; - char *type; - char *alias; - - start = strtok((char *)NULL, WHITE); - end = strtok((char *)NULL, WHITE); - type = strtok((char *)NULL, WHITE); - alias = strtok((char *)NULL, WHITE); - - if (start != (char *)NULL - && end != (char *)NULL - && type != (char *)NULL) { - value *startVal = constVal(start); - value *endVal = constVal(end); - value *aliasVal; - memRange r; - - if (alias == (char *)NULL) { - aliasVal = constVal(0); - } else { - aliasVal = constVal(alias); - } - - r.start_address = (int)floatFromVal(startVal); - r.end_address = (int)floatFromVal(endVal); - r.alias = (int)floatFromVal(aliasVal); - r.bank = (r.start_address >> 7) & 3; - - if (strcmp(type, "RAM") == 0) { - addMemRange(&r, 0); - } else if (strcmp(type, "SFR") == 0) { - addMemRange(&r, 1); - } else { - return 1; - } - } - - return 0; - } else if (startsWith (ptr, "maxram")) { - char *maxRAM = strtok((char *)NULL, WHITE); - - if (maxRAM != (char *)NULL) { - int maxRAMaddress; - value *maxRAMVal; - - maxRAMVal = constVal(maxRAM); - maxRAMaddress = (int)floatFromVal(maxRAMVal); - setMaxRAM(maxRAMaddress); - } - - return 0; - } - return 1; + /* TODO: allow port-specific command line options to specify + * segment names here. + */ + return FALSE; } -static bool -_pic14_parseOptions (int *pargc, char **argv, int *i) +extern set *dataDirsSet; +extern set *includeDirsSet; +/* pic14 port uses include/pic and lib/pic instead of + * include/pic14 and lib/pic14 as indicated by SDCCmain.c's + * setIncludePaths routine. */ +static void +_pic14_initPaths (void) { - /* TODO: allow port-specific command line options to specify - * segment names here. - */ - return FALSE; + char *p; + char *p2=NULL; + set *tempSet=NULL; + + if (options.nostdinc) + return; + + tempSet = appendStrSet(dataDirsSet, NULL, INCLUDE_DIR_SUFFIX DIR_SEPARATOR_STRING "pic"); + mergeSets(&includeDirsSet, tempSet); + + if ((p = getenv(SDCC_INCLUDE_NAME)) != NULL) + { + addSetHead(&includeDirsSet, p); + p2=Safe_alloc(strlen(p)+strlen(DIR_SEPARATOR_STRING)+strlen("pic")+1); + if(p2!=NULL) + { + strcpy(p2, p); + strcat(p2, DIR_SEPARATOR_STRING); + strcat(p2, "pic"); + addSetHead(&includeDirsSet, p2); + } + } } static void _pic14_finaliseOptions (void) { - - port->mem.default_local_map = data; - port->mem.default_globl_map = data; + pCodeInitRegisters(); + + port->mem.default_local_map = data; + port->mem.default_globl_map = data; #if 0 /* Hack-o-matic: if we are using the flat24 model, - * adjust pointer sizes. - */ + * adjust pointer sizes. + */ if (options.model == MODEL_FLAT24) - { - - fprintf (stderr, "*** WARNING: you should use the '-mds390' option " - "for DS80C390 support. This code generator is " - "badly out of date and probably broken.\n"); - - port->s.fptr_size = 3; - port->s.gptr_size = 4; - port->stack.isr_overhead++; /* Will save dpx on ISR entry. */ + { + + fprintf (stderr, "*** WARNING: you should use the '-mds390' option " + "for DS80C390 support. This code generator is " + "badly out of date and probably broken.\n"); + + port->s.fptr_size = 3; + port->s.gptr_size = 4; + port->stack.isr_overhead++; /* Will save dpx on ISR entry. */ #if 1 - port->stack.call_overhead++; /* This acounts for the extra byte - * of return addres on the stack. - * but is ugly. There must be a - * better way. - */ + port->stack.call_overhead++; /* This acounts for the extra byte + * of return addres on the stack. + * but is ugly. There must be a + * better way. + */ #endif - fReturn = fReturn390; - fReturnSize = 5; - } - + fReturn = fReturn390; + fReturnSize = 5; + } + if (options.model == MODEL_LARGE) - { - port->mem.default_local_map = xdata; - port->mem.default_globl_map = xdata; - } + { + port->mem.default_local_map = xdata; + port->mem.default_globl_map = xdata; + } else + { + port->mem.default_local_map = data; + port->mem.default_globl_map = data; + } + + if (options.stack10bit) + { + if (options.model != MODEL_FLAT24) { - port->mem.default_local_map = data; - port->mem.default_globl_map = data; + fprintf (stderr, + "*** warning: 10 bit stack mode is only supported in flat24 model.\n"); + fprintf (stderr, "\t10 bit stack mode disabled.\n"); + options.stack10bit = 0; } - - if (options.stack10bit) + else { - if (options.model != MODEL_FLAT24) - { - fprintf (stderr, - "*** warning: 10 bit stack mode is only supported in flat24 model.\n"); - fprintf (stderr, "\t10 bit stack mode disabled.\n"); - options.stack10bit = 0; - } - else - { - /* Fixup the memory map for the stack; it is now in - * far space and requires a FPOINTER to access it. - */ - istack->fmap = 1; - istack->ptrType = FPOINTER; - } + /* Fixup the memory map for the stack; it is now in + * far space and requires a FPOINTER to access it. + */ + istack->fmap = 1; + istack->ptrType = FPOINTER; } + } #endif } @@ -231,91 +225,124 @@ static void _pic14_genAssemblerPreamble (FILE * of) { char * name = processor_base_name(); - + if(!name) { - - name = "p16f877"; + + name = "16f877"; fprintf(stderr,"WARNING: No Pic has been selected, defaulting to %s\n",name); } - - fprintf (of, "\tlist\tp=%s\n",&name[1]); - fprintf (of, "\t__CONFIG 0x%x\n",getConfigWord(0x2007)); - fprintf (of, "\tradix dec"); - fprintf (of, "\ninclude \"%s.inc\"\n",name); + + fprintf (of, "\tlist\tp=%s\n",name); + fprintf (of, "\tradix dec\n"); + fprintf (of, "\tinclude \"p%s.inc\"\n",name); } /* Generate interrupt vector table. */ static int -_pic14_genIVT (FILE * of, symbol ** interrupts, int maxInterrupts) +_pic14_genIVT (struct dbuf_s * oBuf, symbol ** interrupts, int maxInterrupts) { int i; - + if (options.model != MODEL_FLAT24) - { - /* Let the default code handle it. */ - return FALSE; - } - - fprintf (of, "\t;ajmp\t__sdcc_gsinit_startup\n"); - + { + /* Let the default code handle it. */ + return FALSE; + } + + dbuf_printf (oBuf, "\t;ajmp\t__sdcc_gsinit_startup\n"); + /* now for the other interrupts */ for (i = 0; i < maxInterrupts; i++) + { + if (interrupts[i]) { - if (interrupts[i]) - { - fprintf (of, "\t;ljmp\t%s\n\t.ds\t4\n", interrupts[i]->rname); - } - else - { - fprintf (of, "\t;reti\n\t.ds\t7\n"); - } + dbuf_printf (oBuf, "\t;ljmp\t%s\n\t.ds\t4\n", interrupts[i]->rname); } - + else + { + dbuf_printf (oBuf, "\t;reti\n\t.ds\t7\n"); + } + } + return TRUE; } static bool _hasNativeMulFor (iCode *ic, sym_link *left, sym_link *right) { - sym_link *test = NULL; - value *val; - - fprintf(stderr,"checking for native mult\n"); - +/* +sym_link *test = NULL; +value *val; + */ + + //fprintf(stderr,"checking for native mult\n"); + if ( ic->op != '*') - { - return FALSE; - } + { + return FALSE; + } + + /* multiply chars in-place */ + if (getSize(left) == 1 && getSize(right) == 1) + return TRUE; + + /* use library functions for more complex maths */ + return FALSE; - return TRUE; -/* + /* if ( IS_LITERAL (left)) - { - fprintf(stderr,"left is lit\n"); - test = left; - val = OP_VALUE (IC_LEFT (ic)); - } + { + fprintf(stderr,"left is lit\n"); + test = left; + val = OP_VALUE (IC_LEFT (ic)); + } else if ( IS_LITERAL (right)) - { - fprintf(stderr,"right is lit\n"); - test = left; - val = OP_VALUE (IC_RIGHT (ic)); - } + { + fprintf(stderr,"right is lit\n"); + test = left; + val = OP_VALUE (IC_RIGHT (ic)); + } else + { + fprintf(stderr,"oops, neither is lit so no\n"); + return FALSE; + } + + if ( getSize (test) <= 2) { - fprintf(stderr,"oops, neither is lit so no\n"); - return FALSE; + fprintf(stderr,"yep\n"); + return TRUE; } + fprintf(stderr,"nope\n"); + + return FALSE; + */ +} - if ( getSize (test) <= 2) - { - fprintf(stderr,"yep\n"); - return TRUE; - } - fprintf(stderr,"nope\n"); +/* Indicate which extended bit operations this port supports */ +static bool +hasExtBitOp (int op, int size) +{ + if (op == RRC + || op == RLC + /* || op == GETHBIT */ /* GETHBIT doesn't look complete for PIC */ + ) + return TRUE; + else + return FALSE; +} - return FALSE; -*/ +/* Indicate the expense of an access to an output storage class */ +static int +oclsExpense (struct memmap *oclass) +{ + /* The IN_FARSPACE test is compatible with historical behaviour, */ + /* but I don't think it is applicable to PIC. If so, please feel */ + /* free to remove this test -- EEP */ + if (IN_FARSPACE(oclass)) + return 1; + + return 0; } /** $1 is always the basename. @@ -326,78 +353,154 @@ _hasNativeMulFor (iCode *ic, sym_link *left, sym_link *right) */ static const char *_linkCmd[] = { - "aslink", "-nf", "$1", NULL + "gplink", "$l", "-w", "-r", "-o \"$2\"", "\"$1\"", "$3", NULL }; -/* Sigh. This really is not good. For now, I recommend: - * sdcc -S -mpic14 file.c - * the -S option does not compile or link - */ static const char *_asmCmd[] = { - "gpasm", "-c -I /usr/local/share/gpasm/header", "$1.asm", NULL - + "gpasm", "$l", "$3", "-c", "\"$1.asm\"", NULL + }; +extern set *libFilesSet; +extern set *libDirsSet; +extern set *libPathsSet; +extern set *includeDirsSet; +extern set *userIncDirsSet; +extern set *dataDirsSetSet; +extern set *relFilesSet; +extern set *linkOptionsSet; + +static void _pic14_do_link (void) +{ + hTab *linkValues=NULL; + char lfrm[256]; + char *lcmd; + char temp[128]; + set *tSet=NULL; + int ret; + char * procName; + + /* + * link command format: + * {linker} {incdirs} {lflags} -o {outfile} {spec_ofiles} {ofiles} {libs} + * + */ + + sprintf(lfrm, "{linker} {incdirs} {sysincdirs} {lflags} -w -r -o {outfile} {user_ofile} {spec_ofiles} {ofiles} {libs}"); + + shash_add(&linkValues, "linker", "gplink"); + + /* LIBRARY SEARCH DIRS */ + mergeSets(&tSet, libPathsSet); + mergeSets(&tSet, libDirsSet); + shash_add(&linkValues, "incdirs", joinStrSet(appendStrSet(tSet, "-I\"", "\""))); + + SNPRINTF (&temp[0], 128, "%cpic\"", DIR_SEPARATOR_CHAR); + joinStrSet(appendStrSet(libDirsSet, "-I\"", &temp[0])); + shash_add(&linkValues, "sysincdirs", joinStrSet(appendStrSet(libDirsSet, "-I\"", &temp[0]))); + + shash_add(&linkValues, "lflags", joinStrSet(linkOptionsSet)); + + shash_add(&linkValues, "outfile", fullDstFileName ? fullDstFileName : dstFileName); + + if(fullSrcFileName) { + sprintf(temp, "%s.o", fullDstFileName ? fullDstFileName : dstFileName ); + shash_add(&linkValues, "user_ofile", temp); + } + + shash_add(&linkValues, "ofiles", joinStrSet(relFilesSet)); + + /* LIBRARIES */ + procName = processor_base_name(); + if (!procName) { + procName = "16f877"; + } + + addSet(&libFilesSet, "libsdcc.lib"); + SNPRINTF(&temp[0], 128, "pic%s.lib", procName); + addSet(&libFilesSet, temp); + shash_add(&linkValues, "libs", joinStrSet(libFilesSet)); + + lcmd = msprintf(linkValues, lfrm); + + ret = my_system( lcmd ); + + Safe_free( lcmd ); + + if(ret) + exit(1); +} + /* Globals */ PORT pic_port = { TARGET_ID_PIC, "pic14", - "MCU pic", /* Target name */ - "p16f877", /* Processor */ + "MCU pic", /* Target name */ + "", /* Processor */ { - TRUE, /* Emit glue around main */ + picglue, + TRUE, /* Emit glue around main */ MODEL_SMALL | MODEL_LARGE | MODEL_FLAT24, MODEL_SMALL }, { _asmCmd, NULL, - NULL, - NULL, - //"-plosgffc", /* Options with debug */ - //"-plosgff", /* Options without debug */ + "-g", /* options with --debug */ + NULL, /* options without --debug */ + //"-plosgffc", /* Options with debug */ + //"-plosgff", /* Options without debug */ 0, ".asm", - NULL /* no do_assemble function */ + NULL /* no do_assemble function */ }, { _linkCmd, NULL, - NULL, - ".rel" + _pic14_do_link, /* own do link function */ + ".o", + 0 }, { _defaultRules }, { - /* Sizes: char, short, int, long, ptr, fptr, gptr, bit, float, max */ - 1, 2, 2, 4, 2, 2, 2, 1, 4, 4 - /* TSD - I changed the size of gptr from 3 to 1. However, it should be - 2 so that we can accomodate the PIC's with 4 register banks (like the - 16f877) - */ + /* Sizes: char, short, int, long, ptr, fptr, gptr, bit, float, max */ + 1, 2, 2, 4, 2, 2, 3, 1, 4, 4 + /* TSD - I changed the size of gptr from 3 to 1. However, it should be + 2 so that we can accomodate the PIC's with 4 register banks (like the + 16f877) + */ }, + /* tags for generic pointers */ + { 0x00, 0x00, 0x00, 0x80 }, /* far, near, xstack, code */ { "XSEG (XDATA)", "STACK (DATA)", - "CSEG (CODE)", + "code", "DSEG (DATA)", "ISEG (DATA)", + NULL, /* pdata */ "XSEG (XDATA)", "BSEG (BIT)", "RSEG (DATA)", "GSINIT (CODE)", - "OSEG (OVR,DATA)", + "udata_ovr", "GSFINAL (CODE)", "HOME (CODE)", NULL, // xidata NULL, // xinit + "CONST (CODE)", // const_name - const data (code or not) + "CABS (ABS,CODE)", // cabs_name - const absolute data (code or not) + "XABS (ABS,XDATA)", // xabs_name - absolute xdata + "IABS (ABS,DATA)", // iabs_name - absolute data NULL, NULL, 1 // code is read only }, + { NULL, NULL }, { +1, 1, 4, 1, 1, 0 }, @@ -405,36 +508,55 @@ PORT pic_port = { 1, -1 }, + { + pic14_emitDebuggerSymbol + }, + { + 255/3, /* maxCount */ + 3, /* sizeofElement */ + /* The rest of these costs are bogus. They approximate */ + /* the behavior of src/SDCCicode.c 1.207 and earlier. */ + {4,4,4}, /* sizeofMatchJump[] */ + {0,0,0}, /* sizeofRangeCompare[] */ + 0, /* sizeofSubtract */ + 3, /* sizeofDispatch */ + }, "_", _pic14_init, _pic14_parseOptions, - NULL, + _pic14_poptions, + _pic14_initPaths, _pic14_finaliseOptions, _pic14_setDefaultOptions, pic14_assignRegisters, _pic14_getRegName, _pic14_keywords, _pic14_genAssemblerPreamble, - NULL, /* no genAssemblerEnd */ + NULL, /* no genAssemblerEnd */ _pic14_genIVT, NULL, // _pic14_genXINIT + NULL, /* genInitStartup */ _pic14_reset_regparm, _pic14_regparm, - _process_pragma, /* process a pragma */ + NULL, /* process a pragma */ NULL, _hasNativeMulFor, + hasExtBitOp, /* hasExtBitOp */ + oclsExpense, /* oclsExpense */ FALSE, - 0, /* leave lt */ - 0, /* leave gt */ - 1, /* transform <= to ! > */ - 1, /* transform >= to ! < */ - 1, /* transform != to !(a == b) */ - 0, /* leave == */ - FALSE, /* No array initializer support. */ - 0, /* no CSE cost estimation yet */ - NULL, /* no builtin functions */ - GPOINTER, /* treat unqualified pointers as "generic" pointers */ - 1, /* reset labelKey to 1 */ - 1, /* globals & local static allowed */ +// TRUE, /* little endian */ + FALSE, /* little endian - PIC code enumlates big endian */ + 0, /* leave lt */ + 0, /* leave gt */ + 1, /* transform <= to ! > */ + 1, /* transform >= to ! < */ + 1, /* transform != to !(a == b) */ + 0, /* leave == */ + FALSE, /* No array initializer support. */ + 0, /* no CSE cost estimation yet */ + NULL, /* no builtin functions */ + GPOINTER, /* treat unqualified pointers as "generic" pointers */ + 1, /* reset labelKey to 1 */ + 1, /* globals & local static allowed */ PORT_MAGIC };