X-Git-Url: https://git.gag.com/?a=blobdiff_plain;f=src%2Fmcs51%2Fmain.c;h=bbe4921c6319352360ddfebce789653669531596;hb=022ad5f4b7e62ee89ae9723242d3ecc459391711;hp=eb4843ba52875ff0149dae64e522a7d6ba6d7098;hpb=0e486caba2e97c32e1bf2030d45dddce103beae4;p=fw%2Fsdcc diff --git a/src/mcs51/main.c b/src/mcs51/main.c index eb4843ba..bbe4921c 100644 --- a/src/mcs51/main.c +++ b/src/mcs51/main.c @@ -15,171 +15,131 @@ static char _defaultRules[] = }; /* list of key words used by msc51 */ -static char *_mcs51_keywords[] = { - "at", - "bit", - "code", - "critical", - "data", - "far", - "idata", - "interrupt", - "near", - "pdata", - "reentrant", - "sfr", - "sbit", - "using", - "xdata", - "_data", - "_code", - "_generic", - "_near", - "_xdata", - "_pdata", - "_idata", - NULL +static char *_mcs51_keywords[] = +{ + "at", + "bit", + "code", + "critical", + "data", + "far", + "idata", + "interrupt", + "near", + "pdata", + "reentrant", + "sfr", + "sbit", + "using", + "xdata", + "_data", + "_code", + "_generic", + "_near", + "_xdata", + "_pdata", + "_idata", + "_naked", + NULL }; -void mcs51_assignRegisters (eBBlock **ebbs, int count); +void mcs51_assignRegisters (eBBlock ** ebbs, int count); -static int regParmFlg = 0; /* determine if we can register a parameter */ +static int regParmFlg = 0; /* determine if we can register a parameter */ -static void _mcs51_init(void) +static void +_mcs51_init (void) { - asm_addTree(&asm_asxxxx_mapping); + asm_addTree (&asm_asxxxx_mapping); } -static void _mcs51_reset_regparm() +static void +_mcs51_reset_regparm () { - regParmFlg = 0; + regParmFlg = 0; } -static int _mcs51_regparm( sym_link *l) +static int +_mcs51_regparm (sym_link * l) { - /* for this processor it is simple - can pass only the first parameter in a register */ - if (regParmFlg) - return 0; + /* for this processor it is simple + can pass only the first parameter in a register */ + if (regParmFlg) + return 0; - regParmFlg = 1; - return 1; + regParmFlg = 1; + return 1; } -static bool _mcs51_parseOptions(int *pargc, char **argv, int *i) +static bool +_mcs51_parseOptions (int *pargc, char **argv, int *i) { - /* TODO: allow port-specific command line options to specify - * segment names here. - */ - return FALSE; + /* TODO: allow port-specific command line options to specify + * segment names here. + */ + return FALSE; } -static void _mcs51_finaliseOptions(void) +static void +_mcs51_finaliseOptions (void) { - /* Hack-o-matic: if we are using the flat24 model, - * 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. */ - #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. - */ - #endif - fReturn = fReturn390; - fReturnSize = 5; - } - - if (options.model == MODEL_LARGE) { - 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.model == MODEL_LARGE) { + port->mem.default_local_map = xdata; + port->mem.default_globl_map = xdata; } - - 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; - } + port->mem.default_local_map = data; + port->mem.default_globl_map = data; } } -static void _mcs51_setDefaultOptions(void) +static void +_mcs51_setDefaultOptions (void) { } -static const char *_mcs51_getRegName(struct regs *reg) +static const char * +_mcs51_getRegName (struct regs *reg) { - if (reg) - return reg->name; - return "err"; + if (reg) + return reg->name; + return "err"; } -static void _mcs51_genAssemblerPreamble(FILE *of) +static void +_mcs51_genAssemblerPreamble (FILE * of) { - if (options.model == MODEL_FLAT24) - { - fputs(".flat24 on\t\t; 24 bit flat addressing\n", of); - fputs("dpx = 0x93\t\t; dpx register unknown to assembler\n", of); - fputs("dps = 0x86\t\t; dps register unknown to assembler\n", of); - fputs("dpl1 = 0x84\t\t; dpl1 register unknown to assembler\n", of); - fputs("dph1 = 0x85\t\t; dph1 register unknown to assembler\n", of); - fputs("dpx1 = 0x95\t\t; dpx1 register unknown to assembler\n", of); - } } /* Generate interrupt vector table. */ -static int _mcs51_genIVT(FILE *of, symbol **interrupts, int maxInterrupts) +static int +_mcs51_genIVT (FILE * of, symbol ** interrupts, int maxInterrupts) { - int i; - - if (options.model != MODEL_FLAT24) - { - /* Let the default code handle it. */ - return FALSE; - } - - fprintf (of, "\tajmp\t__sdcc_gsinit_startup\n"); + return FALSE; +} + +/* Do CSE estimation */ +static bool cseCostEstimation (iCode *ic, iCode *pdic) +{ + operand *result = IC_RESULT(ic); + sym_link *result_type = operandType(result); + + /* if it is a pointer then return ok for now */ + if (IC_RESULT(ic) && IS_PTR(result_type)) return 1; - /* now for the other interrupts */ - for (i = 0; i < maxInterrupts; i++) - { - if (interrupts[i]) - { - fprintf(of, "\tljmp\t%s\n\t.ds\t4\n", interrupts[i]->rname); - } - else - { - fprintf(of, "\treti\n\t.ds\t7\n"); - } + /* if bitwise | add & subtract then no since mcs51 is pretty good at it + so we will cse only if they are local (i.e. both ic & pdic belong to + the same basic block */ + if (IS_BITWISE_OP(ic) || ic->op == '+' || ic->op == '-') { + /* then if they are the same Basic block then ok */ + if (ic->eBBlockNum == pdic->eBBlockNum) return 1; + else return 0; } - - return TRUE; + + /* for others it is cheaper to do the cse */ + return 1; } /** $1 is always the basename. @@ -188,84 +148,97 @@ static int _mcs51_genIVT(FILE *of, symbol **interrupts, int maxInterrupts) $l is the list of extra options that should be there somewhere... MUST be terminated with a NULL. */ -static const char *_linkCmd[] = { - "aslink", "-nf", "$1", NULL +static const char *_linkCmd[] = +{ + "{bindir}{sep}aslink", "-nf", "$1", NULL }; -static const char *_asmCmd[] = { - "asx8051", "-plosgffc", "$1.asm", NULL +/* $3 is replaced by assembler.debug_opts resp. port->assembler.plain_opts */ +static const char *_asmCmd[] = +{ + "asx8051", "$l", "$3", "$1.asm", NULL }; /* Globals */ -PORT mcs51_port = { - "mcs51", - "MCU 8051", /* Target name */ - { - TRUE, /* Emit glue around main */ - MODEL_SMALL | MODEL_LARGE | MODEL_FLAT24, - MODEL_SMALL - }, - { - _asmCmd, - "-plosgffc", /* Options with debug */ - "-plosgff", /* Options without debug */ - 0 - }, - { - _linkCmd, - NULL, - ".rel" - }, - { - _defaultRules - }, - { - /* Sizes: char, short, int, long, ptr, fptr, gptr, bit, float, max */ - 1, 1, 2, 4, 1, 2, 3, 1, 4, 4 - }, - { - "XSEG (XDATA)", - "STACK (DATA)", - "CSEG (CODE)", - "DSEG (DATA)", - "ISEG (DATA)", - "XSEG (XDATA)", - "BSEG (BIT)", - "RSEG (DATA)", - "GSINIT (CODE)", - "OSEG (OVR,DATA)", - "GSFINAL (CODE)", - "HOME (CODE)", - NULL, - NULL, - 1 - }, - { - +1, 1, 4, 1, 1, 0 - }, - /* mcs51 has an 8 bit mul */ - { - 1, 0 - }, - "_", - _mcs51_init, - _mcs51_parseOptions, - _mcs51_finaliseOptions, - _mcs51_setDefaultOptions, - mcs51_assignRegisters, - _mcs51_getRegName , - _mcs51_keywords, - _mcs51_genAssemblerPreamble, - _mcs51_genIVT , - _mcs51_reset_regparm, - _mcs51_regparm, +PORT mcs51_port = +{ + TARGET_ID_MCS51, + "mcs51", + "MCU 8051", /* Target name */ + { + TRUE, /* Emit glue around main */ + MODEL_SMALL | MODEL_LARGE, + MODEL_SMALL + }, + { + _asmCmd, + NULL, + "-plosgffc", /* Options with debug */ + "-plosgff", /* Options without debug */ + 0, + ".asm" + }, + { + _linkCmd, + NULL, NULL, - FALSE, - 0, /* leave lt */ - 0, /* leave gt */ - 1, /* transform <= to ! > */ - 1, /* transform >= to ! < */ - 1, /* transform != to !(a == b) */ - 0 /* leave == */ + ".rel" + }, + { + _defaultRules + }, + { + /* Sizes: char, short, int, long, ptr, fptr, gptr, bit, float, max */ + 1, 2, 2, 4, 1, 2, 3, 1, 4, 4 + }, + { + "XSEG (XDATA)", + "STACK (DATA)", + "CSEG (CODE)", + "DSEG (DATA)", + "ISEG (DATA)", + "XSEG (XDATA)", + "BSEG (BIT)", + "RSEG (DATA)", + "GSINIT (CODE)", + "OSEG (OVR,DATA)", + "GSFINAL (CODE)", + "HOME (CODE)", + NULL, + NULL, + 1 + }, + { + +1, 1, 4, 1, 1, 0 + }, + /* mcs51 has an 8 bit mul */ + { + 1, -1 + }, + "_", + _mcs51_init, + _mcs51_parseOptions, + _mcs51_finaliseOptions, + _mcs51_setDefaultOptions, + mcs51_assignRegisters, + _mcs51_getRegName, + _mcs51_keywords, + _mcs51_genAssemblerPreamble, + _mcs51_genIVT, + _mcs51_reset_regparm, + _mcs51_regparm, + NULL, + NULL, + NULL, + 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. */ + cseCostEstimation, + NULL, /* no builtin functions */ + PORT_MAGIC }; -