X-Git-Url: https://git.gag.com/?a=blobdiff_plain;f=src%2FSDCCmain.c;h=2002576ff8f922d86aa5f5941096b1e10f4e6133;hb=b9855fd6b84616c3963897a10fa20ea93531e5ce;hp=502c753ad47e9e8c850e981ec1401e43d4359e22;hpb=df203a96d87ea3322e836e3dd980f64b80b234e7;p=fw%2Fsdcc diff --git a/src/SDCCmain.c b/src/SDCCmain.c index 502c753a..2002576f 100644 --- a/src/SDCCmain.c +++ b/src/SDCCmain.c @@ -22,6 +22,12 @@ what you give them. Help stamp out software-hoarding! -------------------------------------------------------------------------*/ +#ifdef _WIN32 +#include +#else +#include +#endif + #include #include "common.h" #include @@ -31,52 +37,44 @@ #include "MySystem.h" #include "SDCCmacro.h" #include "SDCCutil.h" +#include "SDCCdebug.h" #include "SDCCargs.h" -#if NATIVE_WIN32 +#ifdef _WIN32 #include -#endif - -#if !defined(__BORLANDC__) && !defined(_MSC_VER) +#else #include #include #endif -/** Name of the environment variable checked for other instalations. */ -#define SDCCDIR_NAME "SDCCDIR" - -//REMOVE ME!!! -extern int yyparse (); +/* REMOVE ME!!! */ +extern int yyparse (void); FILE *srcFile; /* source file */ -FILE *cdbFile = NULL; /* debugger information output file */ char *fullSrcFileName; /* full name for the source file; */ /* can be NULL while c1mode or linking without compiling */ char *fullDstFileName; /* full name for the output file; */ /* only given by -o, otherwise NULL */ -size_t fullDstFileNameLen; /* size of previous string. */ char *dstFileName; /* destination file name without extension */ char *dstPath = ""; /* path for the output files; */ /* "" is equivalent with cwd */ char *moduleName; /* module name is source file without path and extension */ - /* can be NULL while linking without compiling */ -const char *preArgv[128]; /* pre-processor arguments */ + /* can be NULL while linking without compiling */ int currRegBank = 0; -int RegBankUsed[4]={1, 0, 0, 0}; /*JCF: Reg Bank 0 used by default*/ +int RegBankUsed[4] = {1, 0, 0, 0}; /*JCF: Reg Bank 0 used by default*/ struct optimize optimize; struct options options; -char *VersionString = SDCC_VERSION_STR; int preProcOnly = 0; int noAssemble = 0; -char *linkOptions[128]; -const char *asmOptions[128]; -char *libFiles[128]; -int nlibFiles = 0; -char *libPaths[128]; -int nlibPaths = 0; -char *relFiles[128]; -int nrelFiles = 0; -static char *preOutName; +set *preArgvSet = NULL; /* pre-processor arguments */ +set *asmOptionsSet = NULL; /* set of assembler options */ +set *linkOptionsSet = NULL; /* set of linker options */ +set *libFilesSet = NULL; +set *libPathsSet = NULL; +set *relFilesSet = NULL; +set *dataDirsSet = NULL; /* list of data search directories */ +set *includeDirsSet = NULL; /* list of include search directories */ +set *libDirsSet = NULL; /* list of lib search directories */ /* uncomment JAMIN_DS390 to always override and use ds390 port for mcs51 work. This is temporary, for compatibility testing. */ @@ -85,13 +83,9 @@ static char *preOutName; int ds390_jammed = 0; #endif -// Globally accessible scratch buffer for file names. +/* Globally accessible scratch buffer for file names. */ char scratchFileName[PATH_MAX]; -char buffer[PATH_MAX]; - -// In MSC VC6 default search path for exe's to path for this - -char DefaultExePath[128]; +char buffer[PATH_MAX * 2]; #define OPTION_HELP "-help" @@ -115,9 +109,10 @@ char DefaultExePath[128]; #define OPTION_DATA_LOC "--data-loc" #define OPTION_CODE_LOC "--code-loc" #define OPTION_IDATA_LOC "--idata-loc" +#define OPTION_NO_LABEL_OPT "--nolabelopt" #define OPTION_NO_LOOP_INV "--noinvariant" #define OPTION_NO_LOOP_IND "--noinduction" -#define OPTION_LESS_PEDANTIC "--lesspedantic" +#define OPTION_LESS_PEDANTIC "--less-pedantic" #define OPTION_NO_GCSE "--nogcse" #define OPTION_SHORT_IS_8BITS "--short-is-8bits" #define OPTION_TINI_LIBID "--tini-libid" @@ -126,6 +121,9 @@ char DefaultExePath[128]; #define OPTION_CODE_SIZE "--code-size" #define OPTION_NO_CCODE_IN_ASM "--no-c-code-in-asm" #define OPTION_ICODE_IN_ASM "--i-code-in-asm" +#define OPTION_PRINT_SEARCH_DIRS "--print-search-dirs" +#define OPTION_MSVC_ERROR_STYLE "--vc" +#define OPTION_USE_STDOUT "--use-stdout" static const OPTION optionsTable[] = { @@ -146,12 +144,17 @@ optionsTable[] = { { 0, OPTION_LARGE_MODEL, NULL, "external data space is used" }, { 0, OPTION_MEDIUM_MODEL, NULL, "not supported" }, { 0, OPTION_SMALL_MODEL, NULL, "internal data space is used (default)" }, +#if !OPT_DISABLE_DS390 { 0, OPTION_FLAT24_MODEL, NULL, "use the flat24 model for the ds390 (default)" }, +#endif { 0, "--stack-auto", &options.stackAuto, "Stack automatic variables" }, +#if !OPT_DISABLE_DS390 { 0, OPTION_STACK_8BIT, NULL, "use the 8bit stack for the ds390 (not supported yet)" }, { 0, "--stack-10bit", &options.stack10bit, "use the 10bit stack for ds390 (default)" }, +#endif { 0, "--xstack", &options.useXstack, "Use external stack" }, { 0, OPTION_NO_GCSE, NULL, "Disable the GCSE optimisation" }, + { 0, OPTION_NO_LABEL_OPT, NULL, "Disable label optimisation" }, { 0, OPTION_NO_LOOP_INV, NULL, "Disable optimisation of invariants" }, { 0, OPTION_NO_LOOP_IND, NULL, NULL }, { 0, "--nojtbound", &optimize.noJTabBoundary, "Don't generate boundary check for jump tables" }, @@ -179,7 +182,7 @@ optionsTable[] = { { 0, OPTION_PEEP_FILE, NULL, " use this extra peep-hole file" }, { 0, OPTION_LIB_PATH, NULL, " use this path to search for libraries" }, { 0, "--int-long-reent", &options.intlong_rent, "Use reenterant calls on the int and long support functions" }, - { 0, "--float-reent", &options.float_rent, "Use reenterant calls on the floar support functions" }, + { 0, "--float-reent", &options.float_rent, "Use reenterant calls on the float support functions" }, { 0, OPTION_OUT_FMT_IHX, NULL, NULL }, { 0, "--out-fmt-s19", &options.out_fmt, NULL }, { 0, "--cyclomatic", &options.cyclomatic, NULL }, @@ -203,14 +206,31 @@ optionsTable[] = { { 0, "--profile", &options.profile, "On supported ports, generate extra profiling information" }, { 0, "--fommit-frame-pointer", &options.ommitFramePtr, "Leave out the frame pointer." }, { 0, "--all-callee-saves", &options.all_callee_saves, "callee will always save registers used" }, +#if !OPT_DISABLE_DS390 { 0, "--use-accelerator", &options.useAccelerator,"generate code for DS390 Arithmetic Accelerator"}, +#endif { 0, "--stack-probe", &options.stack_probe,"insert call to function __stack_probe at each function prologue"}, +#if !OPT_DISABLE_TININative { 0, "--tini-libid", NULL," LibraryID used in -mTININative"}, +#endif +#if !OPT_DISABLE_DS390 { 0, "--protect-sp-update", &options.protect_sp_update,"DS390 - will disable interrupts during ESP:SP updates"}, +#endif +#if !OPT_DISABLE_DS390 || !OPT_DISABLE_MCS51 { 0, "--parms-in-bank1", &options.parms_in_bank1,"MCS51/DS390 - use Bank1 for parameter passing"}, +#endif { 0, OPTION_NO_XINIT_OPT, &options.noXinitOpt, "don't memcpy initialized xram from code"}, { 0, OPTION_NO_CCODE_IN_ASM, &options.noCcodeInAsm, "don't include c-code as comments in the asm file"}, { 0, OPTION_ICODE_IN_ASM, &options.iCodeInAsm, "include i-code as comments in the asm file"}, + { 0, OPTION_PRINT_SEARCH_DIRS, &options.printSearchDirs, "display the directories in the compiler's search path"}, + { 0, OPTION_MSVC_ERROR_STYLE, &options.vc_err_style, "messages are compatible with Micro$oft visual studio"}, + { 0, OPTION_USE_STDOUT, &options.use_stdout, "send errors to stdout instead of stderr"}, +#if !OPT_DISABLE_Z80 || !OPT_DISABLE_GBZ80 + { 0, "--no-std-crt0", &options.no_std_crt0, "For the z80/gbz80 do not link default crt0.o"}, +#endif +#if !OPT_DISABLE_PIC16 + { 0, "--gen-banksel", &options.gen_banksel, "enable the generation of banksel assembler directives in PIC16 port"}, +#endif /* End of options */ { 0, NULL } }; @@ -241,14 +261,14 @@ unsupportedOptTable[] = { /** List of all default constant macros. */ static const char *_baseValues[] = { - "cpp", "{bindir}{sep}sdcpp", + "cpp", "sdcpp", "cppextraopts", "", /* Path seperator character */ "sep", DIR_SEPARATOR_STRING, NULL }; -static const char *_preCmd = "{cpp} -nostdinc -Wall -std=c99 -DSDCC=1 {cppextraopts} {fullsrcfilename} {cppoutfilename}"; +static const char *_preCmd = "{cpp} -nostdinc -Wall -std=c99 -DSDCC=1 {cppextraopts} \"{fullsrcfilename}\" \"{cppoutfilename}\""; PORT *port; @@ -272,20 +292,23 @@ static PORT *_ports[] = #if !OPT_DISABLE_PIC &pic_port, #endif +#if !OPT_DISABLE_PIC16 + &pic16_port, +#endif #if !OPT_DISABLE_TININative &tininative_port, #endif #if !OPT_DISABLE_XA51 &xa51_port, #endif +#if !OPT_DISABLE_DS400 + &ds400_port, +#endif + &hc08_port, }; #define NUM_PORTS (sizeof(_ports)/sizeof(_ports[0])) -#if !OPT_DISABLE_PIC -extern void picglue (); -#endif - /** Sets the port to the one given by the command line option. @param The name minus the option (eg 'mcs51') @return 0 on success. @@ -348,8 +371,15 @@ _findPort (int argc, char **argv) } argv++; } + /* Use the first in the list */ - port = _ports[0]; +#if defined(DEFAULT_PORT) + /* VR - 13/5/2003 DEFAULT_PORT is defined in port.h */ + port = &DEFAULT_PORT; +#else + port = _ports[0]; +#endif + } /* search through the command line options for the processor */ @@ -373,16 +403,20 @@ _findProcessor (int argc, char **argv) /* printVersionInfo - prints the version info */ /*-----------------------------------------------------------------*/ void -printVersionInfo () +printVersionInfo (void) { int i; fprintf (stderr, "SDCC : "); - for (i = 0; i < NUM_PORTS; i++) + for (i = 0; i < NUM_PORTS; i++) { fprintf (stderr, "%s%s", i == 0 ? "" : "/", _ports[i]->target); - - fprintf (stderr, " %s" +#ifdef DEFAULT_PORT + fprintf(stderr, "%s", (&DEFAULT_PORT == _ports[i])?"*":""); +#endif + } + + fprintf (stderr, " " SDCC_VERSION_STR #ifdef SDCC_SUB_VERSION_STR "/" SDCC_SUB_VERSION_STR #endif @@ -390,20 +424,16 @@ printVersionInfo () #ifdef __CYGWIN__ " (CYGWIN)\n" #elif defined __MINGW32__ - " (MINGW32) \n" + " (MINGW32)\n" +#elif defined __DJGPP__ + " (DJGPP)\n" +#elif defined(_MSC_VER) + " (MSVC)\n" +#elif defined(__BORLANDC__) + " (BORLANDC)\n" #else -# ifdef __DJGPP__ - " (DJGPP) \n" -# else -# if defined(_MSC_VER) - " (WIN32) \n" -# else " (UNIX) \n" -# endif -# endif #endif - - ,VersionString ); } @@ -426,7 +456,7 @@ printOptions(const OPTION *optionsTable) /* printUsage - prints command line syntax */ /*-----------------------------------------------------------------*/ void -printUsage () +printUsage (void) { int i; printVersionInfo(); @@ -450,45 +480,33 @@ printUsage () } /*-----------------------------------------------------------------*/ -/* parseWithComma - separates string with comma */ +/* setParseWithComma - separates string with comma to a set */ /*-----------------------------------------------------------------*/ void -parseWithComma (char **dest, char *src) +setParseWithComma (set **dest, char *src) { - int i = 0; + char *p; + int length; - strtok (src, "\r\n \t"); /* skip the initial white spaces */ - while (isspace (*src)) + while (isspace(*src)) src++; - dest[i++] = src; - while (*src) - { - if (*src == ',') - { - *src = '\0'; - src++; - if (*src) - dest[i++] = src; - continue; - } - src++; - } + + /* skip the trailing white spaces */ + length = strlen(src); + while (length && isspace(src[length-1])) + src[--length] = '\0'; + + for (p = strtok(src, ","); p != NULL; p = strtok(NULL, ",")) + addSet(dest, Safe_strdup(p)); } /*-----------------------------------------------------------------*/ /* setDefaultOptions - sets the default options */ /*-----------------------------------------------------------------*/ static void -setDefaultOptions () +setDefaultOptions (void) { - int i; - - for (i = 0; i < 128; i++) - preArgv[i] = asmOptions[i] = - linkOptions[i] = relFiles[i] = libFiles[i] = - libPaths[i] = NULL; - /* first the options part */ options.stack_loc = 0; /* stack pointer initialised to 0 */ options.xstack_loc = 0; /* xternal stack starts at 0 */ @@ -560,7 +578,7 @@ processFile (char *s) } /* copy the file name into the buffer */ - strncpyz (buffer, s, PATH_MAX); + strncpyz (buffer, s, sizeof(buffer)); /* get rid of the "."-extension */ @@ -603,14 +621,14 @@ processFile (char *s) strcmp (fext, ".R") == 0 || strcmp (fext, ".REL") == 0 || strcmp (fext, port->linker.rel_ext) == 0) { - relFiles[nrelFiles++] = s; + addSet(&relFilesSet, Safe_strdup(s)); return; } /* if .lib or .LIB */ if (strcmp (fext, ".lib") == 0 || strcmp (fext, ".LIB") == 0) { - libFiles[nlibFiles++] = s; + addSet(&libFilesSet, Safe_strdup(s)); return; } @@ -633,20 +651,20 @@ _setModel (int model, const char *sz) static char * getStringArg(const char *szStart, char **argv, int *pi, int argc) { - if (argv[*pi][strlen(szStart)]) + if (argv[*pi][strlen(szStart)]) { return &argv[*pi][strlen(szStart)]; } - else + else { ++(*pi); - if (*pi >= argc) + if (*pi >= argc) { werror (E_ARGUMENT_MISSING, szStart); /* Die here rather than checking for errors later. */ exit(-1); } - else + else { return argv[*pi]; } @@ -654,7 +672,7 @@ getStringArg(const char *szStart, char **argv, int *pi, int argc) } /** Gets the integer argument to this option using the same rules as - getStringArg. + getStringArg. */ static int getIntArg(const char *szStart, char **argv, int *pi, int argc) @@ -682,7 +700,7 @@ tryHandleUnsupportedOpt(char **argv, int *pi) if (argv[*pi][1] == '-') { - // Long option. + /* Long option. */ longOpt = argv[*pi]; } else @@ -693,19 +711,22 @@ tryHandleUnsupportedOpt(char **argv, int *pi) { if (unsupportedOptTable[i].shortOpt == shortOpt || (longOpt && unsupportedOptTable[i].longOpt && !strcmp(unsupportedOptTable[i].longOpt, longOpt))) { - // Found an unsupported opt. + /* Found an unsupported opt. */ char buffer[100]; - sprintf(buffer, "%s%c%c", longOpt ? longOpt : "", shortOpt ? '-' : ' ', shortOpt ? shortOpt : ' '); + SNPRINTF(buffer, sizeof(buffer), + "%s%c%c", + longOpt ? longOpt : "", + shortOpt ? '-' : ' ', shortOpt ? shortOpt : ' '); werror (W_UNSUPP_OPTION, buffer, unsupportedOptTable[i].message); return 1; } } - // Didn't find in the table + /* Didn't find in the table */ return 0; } else { - // Not an option, so can't be unsupported :) + /* Not an option, so can't be unsupported :) */ return 0; } } @@ -721,7 +742,7 @@ scanOptionsTable(const OPTION *optionsTable, char shortOpt, const char *longOpt, strcmp(optionsTable[i].longOpt, longOpt) == 0)) { - // If it is a flag then we can handle it here + /* If it is a flag then we can handle it here */ if (optionsTable[i].pparameter != NULL) { if (optionsTable[i].shortOpt == shortOpt) @@ -733,12 +754,12 @@ scanOptionsTable(const OPTION *optionsTable, char shortOpt, const char *longOpt, return 1; } else { - // Not a flag. Handled manually later. + /* Not a flag. Handled manually later. */ return 0; } } } - // Didn't find in the table + /* Didn't find in the table */ return 0; } @@ -752,7 +773,7 @@ tryHandleSimpleOpt(char **argv, int *pi) if (argv[*pi][1] == '-') { - // Long option. + /* Long option. */ longOpt = argv[*pi]; } else @@ -776,7 +797,7 @@ tryHandleSimpleOpt(char **argv, int *pi) } else { - // Not an option, so can't be handled. + /* Not an option, so can't be handled. */ return 0; } } @@ -870,7 +891,7 @@ parseCmdLine (int argc, char **argv) if (strcmp (argv[i], OPTION_LIB_PATH) == 0) { - libPaths[nlibPaths++] = getStringArg(OPTION_LIB_PATH, argv, &i, argc); + addSet(&libPathsSet, Safe_strdup(getStringArg(OPTION_LIB_PATH, argv, &i, argc))); continue; } @@ -883,7 +904,7 @@ parseCmdLine (int argc, char **argv) if (strcmp (argv[i], OPTION_CALLEE_SAVES) == 0) { - parseWithComma (options.calleeSaves, getStringArg(OPTION_CALLEE_SAVES, argv, &i, argc)); + setParseWithComma(&options.calleeSavesSet, getStringArg(OPTION_CALLEE_SAVES, argv, &i, argc)); continue; } @@ -954,6 +975,12 @@ parseCmdLine (int argc, char **argv) continue; } + if (strcmp (argv[i], OPTION_NO_LABEL_OPT) == 0) + { + optimize.label4 = 0; + continue; + } + if (strcmp (argv[i], OPTION_NO_LOOP_IND) == 0) { optimize.loopInduction = 0; @@ -1017,11 +1044,11 @@ parseCmdLine (int argc, char **argv) break; case 'L': - libPaths[nlibPaths++] = getStringArg("-L", argv, &i, argc); + addSet(&libPathsSet, Safe_strdup(getStringArg("-L", argv, &i, argc))); break; case 'l': - libFiles[nlibFiles++] = getStringArg("-l", argv, &i, argc); + addSet(&libFilesSet, Safe_strdup(getStringArg("-l", argv, &i, argc))); break; case 'o': @@ -1029,7 +1056,8 @@ parseCmdLine (int argc, char **argv) char *p; /* copy the file name into the buffer */ - strncpyz(buffer, getStringArg("-o", argv, &i, argc), PATH_MAX); + strncpyz(buffer, getStringArg("-o", argv, &i, argc), + sizeof(buffer)); /* point to last character */ p = buffer + strlen (buffer) - 1; if (*p == DIR_SEPARATOR_CHAR) @@ -1041,7 +1069,6 @@ parseCmdLine (int argc, char **argv) else { fullDstFileName = Safe_strdup (buffer); - fullDstFileNameLen = strlen(fullDstFileName) + 1; /* get rid of the "."-extension */ @@ -1069,17 +1096,17 @@ parseCmdLine (int argc, char **argv) /* pre-processer options */ if (argv[i][2] == 'p') { - parseWithComma ((char **)preArgv, getStringArg("-Wp", argv, &i, argc)); + setParseWithComma(&preArgvSet, getStringArg("-Wp", argv, &i, argc)); } /* linker options */ else if (argv[i][2] == 'l') { - parseWithComma(linkOptions, getStringArg("-Wl", argv, &i, argc)); + setParseWithComma(&linkOptionsSet, getStringArg("-Wl", argv, &i, argc)); } /* assembler options */ else if (argv[i][2] == 'a') { - parseWithComma ((char **) asmOptions, getStringArg("-Wa", argv, &i, argc)); + setParseWithComma(&asmOptionsSet, getStringArg("-Wa", argv, &i, argc)); } else { @@ -1098,14 +1125,15 @@ parseCmdLine (int argc, char **argv) case 'M': { preProcOnly = 1; - addToList (preArgv, "-M"); + addSet(&preArgvSet, Safe_strdup("-M")); break; } case 'C': { - addToList (preArgv, "-C"); + addSet(&preArgvSet, Safe_strdup("-C")); break; } + case 'd': case 'D': case 'I': @@ -1135,8 +1163,9 @@ parseCmdLine (int argc, char **argv) if (sOpt == 'Y') sOpt = 'I'; - sprintf (buffer, "-%c%s", sOpt, rest); - addToList (preArgv, buffer); + SNPRINTF (buffer, sizeof(buffer), + ((sOpt == 'I') ? "-%c\"%s\"": "-%c%s"), sOpt, rest); + addSet(&preArgvSet, Safe_strdup(buffer)); } break; @@ -1164,7 +1193,7 @@ parseCmdLine (int argc, char **argv) /* some sanity checks in c1 mode */ if (options.c1mode) { - int i; + const char *s; if (fullSrcFileName) { @@ -1172,16 +1201,18 @@ parseCmdLine (int argc, char **argv) werror (W_NO_FILE_ARG_IN_C1, fullSrcFileName); } fullSrcFileName = NULL; - for (i = 0; i < nrelFiles; ++i) + for (s = setFirstItem(relFilesSet); s != NULL; s = setNextItem(relFilesSet)) { - werror (W_NO_FILE_ARG_IN_C1, relFiles[i]); + werror (W_NO_FILE_ARG_IN_C1, s); } - for (i = 0; i < nlibFiles; ++i) + for (s = setFirstItem(libFilesSet); s != NULL; s = setNextItem(libFilesSet)) { - werror (W_NO_FILE_ARG_IN_C1, libFiles[i]); + werror (W_NO_FILE_ARG_IN_C1, s); } - nrelFiles = nlibFiles = 0; - if (options.cc_only || noAssemble || preProcOnly) + deleteSet(&relFilesSet); + deleteSet(&libFilesSet); + + if (options.cc_only || noAssemble || preProcOnly) { werror (W_ILLEGAL_OPT_COMBINATION); } @@ -1195,6 +1226,8 @@ parseCmdLine (int argc, char **argv) /* if no dstFileName given with -o, we've to find one: */ if (!dstFileName) { + const char *s; + /* use the modulename from the C-source */ if (fullSrcFileName) { @@ -1205,12 +1238,12 @@ parseCmdLine (int argc, char **argv) strncatz (dstFileName, moduleName, bufSize); } /* use the modulename from the first object file */ - else if (nrelFiles >= 1) + else if ((s = peekSet(relFilesSet)) != NULL) { char *objectName; size_t bufSize; - strncpyz (buffer, relFiles[0], PATH_MAX); + strncpyz (buffer, s, sizeof(buffer)); /* remove extension (it must be .rel) */ *strrchr (buffer, '.') = '\0'; /* remove path */ @@ -1235,18 +1268,19 @@ parseCmdLine (int argc, char **argv) if (!options.xstack_loc) options.xstack_loc = options.xdata_loc; - /* if debug option is set the open the cdbFile */ + /* if debug option is set then open the cdbFile */ if (options.debug && fullSrcFileName) { - sprintf (scratchFileName, "%s.adb", dstFileName); //JCF: Nov 30, 2002 - if ((cdbFile = fopen (scratchFileName, "w")) == NULL) - werror (E_FILE_OPEN_ERR, scratchFileName); + SNPRINTF (scratchFileName, sizeof(scratchFileName), + "%s.adb", dstFileName); /*JCF: Nov 30, 2002*/ + if(debugFile->openFile(scratchFileName)) + debugFile->writeModule(moduleName); else - { - /* add a module record */ - fprintf (cdbFile, "M:%s\n", moduleName); - } + werror (E_FILE_OPEN_ERR, scratchFileName); } + MSVC_style(options.vc_err_style); + if(options.use_stdout) dup2(STDOUT_FILENO, STDERR_FILENO); + return 0; } @@ -1258,33 +1292,45 @@ linkEdit (char **envp) { FILE *lnkfile; char *segName, *c; - int i, system_ret; + int system_ret; + const char *s; /* first we need to create the .lnk file */ - sprintf (scratchFileName, "%s.lnk", dstFileName); + SNPRINTF (scratchFileName, sizeof(scratchFileName), + "%s.lnk", dstFileName); if (!(lnkfile = fopen (scratchFileName, "w"))) { werror (E_FILE_OPEN_ERR, scratchFileName); exit (1); } - /* now write the options. JCF: added option 'y' */ - fprintf (lnkfile, "-myux%c\n", (options.out_fmt ? 's' : 'i')); - - /* if iram size specified */ - if (options.iram_size) - fprintf (lnkfile, "-a 0x%04x\n", options.iram_size); - - /* if xram size specified */ - if (options.xram_size_set) - fprintf (lnkfile, "-v 0x%04x\n", options.xram_size); - - /* if code size specified */ - if (options.code_size) - fprintf (lnkfile, "-w 0x%04x\n", options.code_size); - - if (options.debug) - fprintf (lnkfile, "-z\n"); + if (TARGET_IS_Z80 || TARGET_IS_GBZ80) + { + fprintf (lnkfile, "--\n-m\n-j\n-x\n-%c %s\n", + (options.out_fmt ? 's' : 'i'), dstFileName); + } + else /*For all the other ports. Including pics???*/ + { + fprintf (lnkfile, "-myux%c\n", (options.out_fmt ? 's' : 'i')); + } + + if (!(TARGET_IS_Z80 || TARGET_IS_GBZ80)) /*Not for the z80, gbz80*/ + { + /* if iram size specified */ + if (options.iram_size) + fprintf (lnkfile, "-a 0x%04x\n", options.iram_size); + + /* if xram size specified */ + if (options.xram_size_set) + fprintf (lnkfile, "-v 0x%04x\n", options.xram_size); + + /* if code size specified */ + if (options.code_size) + fprintf (lnkfile, "-w 0x%04x\n", options.code_size); + + if (options.debug) + fprintf (lnkfile, "-z\n"); + } #define WRITE_SEG_LOC(N, L) \ segName = Safe_strdup(N); \ @@ -1292,95 +1338,201 @@ linkEdit (char **envp) fprintf (lnkfile,"-b %s = 0x%04x\n", c, L); \ if (segName) { Safe_free(segName); } - /* code segment start */ - WRITE_SEG_LOC (CODE_NAME, options.code_loc); + if (!(TARGET_IS_Z80 || TARGET_IS_GBZ80)) /*Not for the z80, gbz80*/ + { - /* data segment start */ - if(options.data_loc){ /*JCF: If zero, the linker chooses the best place for data*/ - WRITE_SEG_LOC (DATA_NAME, options.data_loc); - } + /* code segment start */ + WRITE_SEG_LOC (CODE_NAME, options.code_loc); - /* xdata start */ - WRITE_SEG_LOC (XDATA_NAME, options.xdata_loc); + /* data segment start. If zero, the linker chooses + the best place for data*/ + if(options.data_loc){ + WRITE_SEG_LOC (DATA_NAME, options.data_loc); + } - /* indirect data */ - if (IDATA_NAME) { - WRITE_SEG_LOC (IDATA_NAME, options.idata_loc); - } + /* xdata start */ + WRITE_SEG_LOC (XDATA_NAME, options.xdata_loc); - /* bit segment start */ - WRITE_SEG_LOC (BIT_NAME, 0); + /* indirect data */ + if (IDATA_NAME) { + WRITE_SEG_LOC (IDATA_NAME, options.idata_loc); + } + + /* bit segment start */ + WRITE_SEG_LOC (BIT_NAME, 0); - /* JCF: stack start */ - if ( (options.stack_loc) && (options.stack_loc<0x100) ) { - WRITE_SEG_LOC ("SSEG", options.stack_loc); + /* stack start */ + if ( (options.stack_loc) && (options.stack_loc<0x100) ) { + WRITE_SEG_LOC ("SSEG", options.stack_loc); + } + } + else /*For the z80, gbz80*/ + { + WRITE_SEG_LOC ("_CODE", options.code_loc); + WRITE_SEG_LOC ("_DATA", options.data_loc); + } + + /* If the port has any special linker area declarations, get 'em */ + if (port->extraAreas.genExtraAreaLinkOptions) + { + port->extraAreas.genExtraAreaLinkOptions(lnkfile); } /* add the extra linker options */ - for (i = 0; linkOptions[i]; i++) - fprintf (lnkfile, "%s\n", linkOptions[i]); + fputStrSet(lnkfile, linkOptionsSet); /* other library paths if specified */ - for (i = 0; i < nlibPaths; i++) - fprintf (lnkfile, "-k %s\n", libPaths[i]); - + for (s = setFirstItem(libPathsSet); s != NULL; s = setNextItem(libPathsSet)) + fprintf (lnkfile, "-k %s\n", s); + /* standard library path */ - if (!options.nostdlib) + if (!options.nostdlib) { - switch (options.model) - { - case MODEL_SMALL: - c = "small"; - break; - case MODEL_LARGE: - c = "large"; - break; - case MODEL_FLAT24: - /* c = "flat24"; */ - c = "ds390"; - break; - case MODEL_PAGE0: - c = "xa51"; - break; - default: - werror (W_UNKNOWN_MODEL, __FILE__, __LINE__); - c = "unknown"; - break; - } - mfprintf (lnkfile, getRuntimeVariables(), "-k {libdir}{sep}%s\n", c); + if (!(TARGET_IS_Z80 || TARGET_IS_GBZ80 || TARGET_IS_HC08)) /*Not for the z80, gbz80*/ + { + switch (options.model) + { + case MODEL_SMALL: + c = "small"; + break; + case MODEL_LARGE: + c = "large"; + break; + case MODEL_FLAT24: + /* c = "flat24"; */ + if (TARGET_IS_DS390) + { + c = "ds390"; + } + else if (TARGET_IS_DS400) + { + c = "ds400"; + } + else + { + fprintf(stderr, + "Add support for your FLAT24 target in %s @ line %d\n", + __FILE__, __LINE__); + exit(-1); + } + break; + case MODEL_PAGE0: + c = "xa51"; + break; + default: + werror (W_UNKNOWN_MODEL, __FILE__, __LINE__); + c = "unknown"; + break; + } + } + else /*for the z80, gbz80*/ + { + if (TARGET_IS_HC08) + c = "hc08"; + else if (TARGET_IS_Z80) + c = "z80"; + else + c = "gbz80"; + } + for (s = setFirstItem(libDirsSet); s != NULL; s = setNextItem(libDirsSet)) + mfprintf (lnkfile, getRuntimeVariables(), "-k %s{sep}%s\n", s, c); + /* standard library files */ #if !OPT_DISABLE_DS390 - if (options.model == MODEL_FLAT24) - { - fprintf (lnkfile, "-l %s\n", STD_DS390_LIB); - } + if (options.model == MODEL_FLAT24) + { + if (TARGET_IS_DS390) + { + fprintf (lnkfile, "-l %s\n", STD_DS390_LIB); + } + else if (TARGET_IS_DS400) + { + fprintf (lnkfile, "-l %s\n", STD_DS400_LIB); + } + else + { + fprintf(stderr, + "Add support for your FLAT24 target in %s @ line %d\n", + __FILE__, __LINE__); + exit(-1); + } + } #endif #if !OPT_DISABLE_XA51 #ifdef STD_XA51_LIB - if (options.model == MODEL_PAGE0) - { - fprintf (lnkfile, "-l %s\n", STD_XA51_LIB); - } + if (options.model == MODEL_PAGE0) + { + fprintf (lnkfile, "-l %s\n", STD_XA51_LIB); + } #endif #endif - fprintf (lnkfile, "-l %s\n", STD_LIB); - fprintf (lnkfile, "-l %s\n", STD_INT_LIB); - fprintf (lnkfile, "-l %s\n", STD_LONG_LIB); - fprintf (lnkfile, "-l %s\n", STD_FP_LIB); + if (!(TARGET_IS_Z80 || TARGET_IS_GBZ80 + || TARGET_IS_HC08)) /*Not for the z80, gbz80*/ + { /*Why the z80 port is not using the standard libraries?*/ + fprintf (lnkfile, "-l %s\n", STD_LIB); + fprintf (lnkfile, "-l %s\n", STD_INT_LIB); + fprintf (lnkfile, "-l %s\n", STD_LONG_LIB); + fprintf (lnkfile, "-l %s\n", STD_FP_LIB); + } + else if (TARGET_IS_HC08) + { + fprintf (lnkfile, "-l hc08\n"); + } + else if (TARGET_IS_Z80) + { + fprintf (lnkfile, "-l z80\n"); + } + else if (TARGET_IS_GBZ80) + { + fprintf (lnkfile, "-l gbz80\n"); + } } /* additional libraries if any */ - for (i = 0; i < nlibFiles; i++) - fprintf (lnkfile, "-l %s\n", libFiles[i]); + for (s = setFirstItem(libFilesSet); s != NULL; s = setNextItem(libFilesSet)) + fprintf (lnkfile, "-l %s\n", s); + + /*For the z80 and gbz80 ports, try to find where crt0.o is... + It is very important for this file to be first on the linking proccess + so the areas are set in the correct order, expecially _GSINIT*/ + if ((TARGET_IS_Z80 || TARGET_IS_GBZ80) && + !options.no_std_crt0 && !options.nostdlib) /*For the z80, gbz80*/ + { + char crt0path[PATH_MAX]; + FILE * crt0fp; + for (s = setFirstItem(libDirsSet); s != NULL; s = setNextItem(libDirsSet)) + { + sprintf (crt0path, "%s%s%s%scrt0.o", + s, DIR_SEPARATOR_STRING, c, DIR_SEPARATOR_STRING); + + crt0fp=fopen(crt0path, "r"); + if(crt0fp!=NULL)/*Found it!*/ + { + fclose(crt0fp); + #ifdef __CYGWIN__ + { + /*The CYGWIN version of the z80-gbz80 linker is getting confused with + windows paths, so convert them to the CYGWIN format*/ + char posix_path[PATH_MAX]; + void cygwin_conv_to_full_posix_path(char * win_path, char * posix_path); + cygwin_conv_to_full_posix_path(crt0path, posix_path); + strcpy(crt0path, posix_path); + } + #endif + fprintf (lnkfile, "%s\n", crt0path); + break; + } + } + if(s==NULL) fprintf (stderr, "Warning: couldn't find crt0.o\n"); + } /* put in the object files */ if (fullSrcFileName) - fprintf (lnkfile, "%s ", dstFileName); + fprintf (lnkfile, "%s%s\n", dstFileName, port->linker.rel_ext); - for (i = 0; i < nrelFiles; i++) - fprintf (lnkfile, "%s\n", relFiles[i]); + fputStrSet(lnkfile, relFilesSet); fprintf (lnkfile, "\n-e\n"); fclose (lnkfile); @@ -1393,35 +1545,46 @@ linkEdit (char **envp) /* -o option overrides default name? */ if (fullDstFileName) { - strncpyz (scratchFileName, fullDstFileName, PATH_MAX); + strncpyz (scratchFileName, fullDstFileName, sizeof(scratchFileName)); } else { /* the linked file gets the name of the first modul */ if (fullSrcFileName) { - strncpyz (scratchFileName, dstFileName, PATH_MAX); + strncpyz (scratchFileName, dstFileName, sizeof(scratchFileName)); } else { - strncpyz (scratchFileName, relFiles[0], PATH_MAX); + s = peekSet(relFilesSet); + + assert(s); + + strncpyz (scratchFileName, s, sizeof(scratchFileName)); /* strip ".rel" extension */ *strrchr (scratchFileName, '.') = '\0'; } - strncatz (scratchFileName, options.out_fmt ? ".S19" : ".ihx", PATH_MAX); + strncatz (scratchFileName, + options.out_fmt ? ".S19" : ".ihx", + sizeof(scratchFileName)); } if (port->linker.cmd) { char buffer2[PATH_MAX]; - buildCmdLine (buffer2, port->linker.cmd, dstFileName, scratchFileName, NULL, NULL); - buildCmdLine2 (buffer, buffer2, PATH_MAX); + + /* VR 030517 - gplink needs linker options to set the linker script,*/ + buildCmdLine (buffer2, port->linker.cmd, dstFileName, scratchFileName, NULL, linkOptionsSet); + + buildCmdLine2 (buffer, sizeof(buffer), buffer2); } else { - buildCmdLine2 (buffer, port->linker.mcmd, PATH_MAX); + buildCmdLine2 (buffer, sizeof(buffer), port->linker.mcmd); } +/* if (options.verbose)fprintf(stderr, "linker command line: %s\n", buffer); */ + system_ret = my_system (buffer); /* TODO: most linker don't have a -o parameter */ /* -o option overrides default name? */ @@ -1431,47 +1594,46 @@ linkEdit (char **envp) /* the linked file gets the name of the first modul */ if (fullSrcFileName) { - strncpyz (scratchFileName, dstFileName, PATH_MAX); + strncpyz (scratchFileName, dstFileName, sizeof(scratchFileName)); p = strlen (scratchFileName) + scratchFileName; } else { - strncpyz (scratchFileName, relFiles[0], PATH_MAX); - /* strip "rel" extension */ + s = peekSet(relFilesSet); + + assert(s); + + strncpyz (scratchFileName, s, sizeof(scratchFileName)); + /* strip ".rel" extension */ p = strrchr (scratchFileName, '.'); if (p) { - p++; - *p = 0; + *p = 0; } - } - strncatz (scratchFileName, options.out_fmt ? "S19" : "ihx", PATH_MAX); + strncatz (scratchFileName, + options.out_fmt ? ".S19" : ".ihx", + sizeof(scratchFileName)); rename (scratchFileName, fullDstFileName); - q = strrchr (fullDstFileName, '.'); - if (q) - { - /* point after the '.' of the extension */ - q++; - } - else + strncpyz (buffer, fullDstFileName, sizeof(buffer)); + q = strrchr (buffer, '.'); + if (!q) { /* no extension: append new extensions */ - /* Don't we want to append a period here ? */ - q = strlen (fullDstFileName) + fullDstFileName; + q = strlen (buffer) + buffer; } - - *p = 0; - strncatz (scratchFileName, "map", PATH_MAX); + + *p = 0; + strncatz (scratchFileName, ".map", sizeof(scratchFileName)); *q = 0; - strncatz(fullDstFileName, "map", PATH_MAX); - rename (scratchFileName, fullDstFileName); - *p = 0; - strncatz (scratchFileName, "mem", PATH_MAX); + strncatz(buffer, ".map", sizeof(buffer)); + rename (scratchFileName, buffer); + *p = 0; + strncatz (scratchFileName, ".mem", sizeof(scratchFileName)); *q = 0; - strncatz(fullDstFileName, "mem", PATH_MAX); - rename (scratchFileName, fullDstFileName); + strncatz(buffer, ".mem", sizeof(buffer)); + rename (scratchFileName, buffer); } if (system_ret) { @@ -1489,22 +1651,23 @@ assemble (char **envp) /* -o option overrides default name? */ if (options.cc_only && fullDstFileName) { - strncpyz (scratchFileName, fullDstFileName, PATH_MAX); + strncpyz (scratchFileName, fullDstFileName, sizeof(scratchFileName)); } else { /* the assembled file gets the name of the first modul */ - strncpyz (scratchFileName, dstFileName, PATH_MAX); - strncatz (scratchFileName, port->linker.rel_ext, PATH_MAX); + strncpyz (scratchFileName, dstFileName, sizeof(scratchFileName)); + strncatz (scratchFileName, port->linker.rel_ext, + sizeof(scratchFileName)); } if (port->assembler.do_assemble) { - port->assembler.do_assemble(asmOptions); + port->assembler.do_assemble(asmOptionsSet); return ; } else if (port->assembler.cmd) { buildCmdLine (buffer, port->assembler.cmd, dstFileName, scratchFileName, options.debug ? port->assembler.debug_opts : port->assembler.plain_opts, - asmOptions); + asmOptionsSet); } else { - buildCmdLine2 (buffer, port->assembler.mcmd, PATH_MAX); + buildCmdLine2 (buffer, sizeof(buffer), port->assembler.mcmd); } if (my_system (buffer)) { @@ -1516,8 +1679,10 @@ assemble (char **envp) /* TODO: most assembler don't have a -o parameter */ /* -o option overrides default name? */ if (options.cc_only && fullDstFileName) { - strncpyz (scratchFileName, dstFileName, PATH_MAX); - strncatz (scratchFileName, port->linker.rel_ext, PATH_MAX); + strncpyz (scratchFileName, dstFileName, sizeof(scratchFileName)); + strncatz (scratchFileName, + port->linker.rel_ext, + sizeof(scratchFileName)); rename (scratchFileName, fullDstFileName); } } @@ -1534,42 +1699,45 @@ preProcess (char **envp) } else { + const char *s; + set *inclList = NULL; + /* if using external stack define the macro */ if (options.useXstack) - addToList (preArgv, "-DSDCC_USE_XSTACK"); + addSet(&preArgvSet, Safe_strdup("-DSDCC_USE_XSTACK")); /* set the macro for stack autos */ if (options.stackAuto) - addToList (preArgv, "-DSDCC_STACK_AUTO"); + addSet(&preArgvSet, Safe_strdup("-DSDCC_STACK_AUTO")); /* set the macro for stack autos */ if (options.stack10bit) - addToList (preArgv, "-DSDCC_STACK_TENBIT"); + addSet(&preArgvSet, Safe_strdup("-DSDCC_STACK_TENBIT")); /* set the macro for no overlay */ if (options.noOverlay) - addToList (preArgv, "-DSDCC_NOOVERLAY"); + addSet(&preArgvSet, Safe_strdup("-DSDCC_NOOVERLAY")); /* set the macro for large model */ switch (options.model) { case MODEL_LARGE: - addToList (preArgv, "-DSDCC_MODEL_LARGE"); + addSet(&preArgvSet, Safe_strdup("-DSDCC_MODEL_LARGE")); break; case MODEL_SMALL: - addToList (preArgv, "-DSDCC_MODEL_SMALL"); + addSet(&preArgvSet, Safe_strdup("-DSDCC_MODEL_SMALL")); break; case MODEL_COMPACT: - addToList (preArgv, "-DSDCC_MODEL_COMPACT"); + addSet(&preArgvSet, Safe_strdup("-DSDCC_MODEL_COMPACT")); break; case MODEL_MEDIUM: - addToList (preArgv, "-DSDCC_MODEL_MEDIUM"); + addSet(&preArgvSet, Safe_strdup("-DSDCC_MODEL_MEDIUM")); break; case MODEL_FLAT24: - addToList (preArgv, "-DSDCC_MODEL_FLAT24"); + addSet(&preArgvSet, Safe_strdup("-DSDCC_MODEL_FLAT24")); break; case MODEL_PAGE0: - addToList (preArgv, "-DSDCC_MODEL_PAGE0"); + addSet(&preArgvSet, Safe_strdup("-DSDCC_MODEL_PAGE0")); break; default: werror (W_UNKNOWN_MODEL, __FILE__, __LINE__); @@ -1577,182 +1745,180 @@ preProcess (char **envp) } /* add port (processor information to processor */ - addToList (preArgv, "-DSDCC_{port}"); - addToList (preArgv, "-D__{port}"); + addSet(&preArgvSet, Safe_strdup("-DSDCC_{port}")); + addSet(&preArgvSet, Safe_strdup("-D__{port}")); /* standard include path */ if (!options.nostdinc) { - addToList (preArgv, "-I{includedir}"); + inclList = appendStrSet(includeDirsSet, "-I\"", "\""); + mergeSets(&preArgvSet, inclList); } - setMainValue ("cppextraopts", join(preArgv)); + setMainValue("cppextraopts", (s = joinStrSet(preArgvSet))); + Safe_free((void *)s); + if (inclList != NULL) + deleteSet(&inclList); - if (preProcOnly) - { - if (fullDstFileName) - { - preOutName = Safe_strdup (fullDstFileName); - } - } + if (preProcOnly && fullDstFileName) + { + /* -E and -o given */ + setMainValue ("cppoutfilename", fullDstFileName); + } else - preOutName = Safe_strdup (tempfilename ()); - - /* Have to set cppoutfilename to something, even if just pre-processing. */ - setMainValue ("cppoutfilename", preOutName ? preOutName : ""); - addSetHead (&tmpfileNameSet, preOutName); + { + /* Piping: set cppoutfilename to NULL, to avoid empty quotes */ + setMainValue ("cppoutfilename", NULL); + } if (options.verbose) printf ("sdcc: Calling preprocessor...\n"); - buildCmdLine2 (buffer, _preCmd, PATH_MAX); + buildCmdLine2 (buffer, sizeof(buffer), _preCmd); - if (my_system (buffer)) - { + if (preProcOnly) { + if (my_system (buffer)) { exit (1); } - if (preProcOnly) - { - exit (0); - } - yyin = fopen (preOutName, "r"); - if (yyin == NULL) - { - perror ("Preproc file not found\n"); + + exit (0); + } + + yyin = my_popen (buffer); + if (yyin == NULL) { + perror ("Preproc file not found"); exit (1); - } - addSetHead (&tmpfileSet, yyin); + } + addSetHead (&pipeSet, yyin); } return 0; } -static bool -_setPaths (const char *pprefix) +/* Set bin paths */ +static void +setBinPaths(const char *argv0) { - /* Logic: - Given the prefix and how the directories were layed out at - configure time, see if the library and include directories are - where expected. If so, set. - */ - getPathDifference (buffer, PREFIX, SDCC_INCLUDE_DIR); - strncpyz (scratchFileName, pprefix, PATH_MAX); - strncatz (scratchFileName, buffer, PATH_MAX); - - if (pathExists (scratchFileName)) - { - setMainValue ("includedir", scratchFileName); - } - else - { - return FALSE; - } + char *p; + char buf[PATH_MAX]; + + /* + * Search logic: + * + * 1. - $SDCCDIR/PREFIX2BIN_DIR + * 2. - path(argv[0]) + * 3. - $PATH + */ - getPathDifference (buffer, PREFIX, SDCC_LIB_DIR); - strncpyz (scratchFileName, pprefix, PATH_MAX); - strncatz (scratchFileName, buffer, PATH_MAX); + /* do it in reverse mode, so that addSetHead() can be used + instead of slower addSet() */ - if (pathExists (scratchFileName)) - { - setMainValue ("libdir", scratchFileName); - } - else - { - return FALSE; - } + if ((p = getBinPath(argv0)) != NULL) + addSetHead(&binPathSet, Safe_strdup(p)); + + if ((p = getenv(SDCC_DIR_NAME)) != NULL) { + SNPRINTF(buf, sizeof buf, "%s" PREFIX2BIN_DIR, p); + addSetHead(&binPathSet, Safe_strdup(buf)); + } - return TRUE; + if (options.printSearchDirs) { + printf("programs:\n"); + fputStrSet(stdout, binPathSet); + } } +/* Set system include path */ static void -_discoverPaths (const char *argv0) +setIncludePath(void) { - /* Logic: - 1. Try the SDCCDIR environment variable. - 2. If (1) fails, and if the argv[0] includes a path, attempt to find the include - and library paths with respect to that. Note that under win32 - argv[0] is always the full path to the program. - 3. If (1) and (2) fail, fall back to the compile time defaults. - - Detecting assumes the same layout as when configured. If the - directories have been further moved about then discovery will - fail. - */ - - /* Some input cases: - "c:\fish\sdcc\bin\sdcc" - "../bin/sdcc" - "/home/fish/bin/sdcc" - - Note that ./sdcc is explicitly not supported as there isn't - enough information. - */ - /* bindir is handled differently to the lib and include directories. - It's rather unfortunate, but required due to the different - install and development layouts. Logic is different as well. - Sigh. + char *p; + + /* + * Search logic: + * + * 1. - $SDCC_INCLUDE + * 2. - $SDCC_HOME/PREFIX2DATA_DIR/INCLUDE_DIR_SUFFIX + * 3. - path(argv[0])/BIN2DATA_DIR/INCLUDE_DIR_SUFFIX + * 4. - DATADIR/INCLUDE_DIR_SUFFIX (only on *nix) */ - if (strchr (argv0, DIR_SEPARATOR_CHAR)) - { - strncpyz (scratchFileName, argv0, PATH_MAX); - *strrchr (scratchFileName, DIR_SEPARATOR_CHAR) = '\0'; - setMainValue ("bindir", scratchFileName); - ExePathList[0] = Safe_strdup (scratchFileName); - } - else if (getenv (SDCCDIR_NAME) != NULL) - { - getPathDifference (buffer, PREFIX, BINDIR); - strncpyz (scratchFileName, getenv (SDCCDIR_NAME), PATH_MAX); - strncatz (scratchFileName, buffer, PATH_MAX); - setMainValue ("bindir", scratchFileName); - ExePathList[0] = Safe_strdup (scratchFileName); - } - else - { - setMainValue ("bindir", BINDIR); - ExePathList[0] = BINDIR; - } - do - { - /* Case 1 */ - if (getenv (SDCCDIR_NAME) != NULL) - { - if (_setPaths (getenv (SDCCDIR_NAME))) - { - /* Successfully set. */ - break; - } - else - { - /* Include and lib weren't where expected. */ - } - } - /* Case 2 */ - if (strchr (argv0, DIR_SEPARATOR_CHAR)) - { - char *pbase = getPrefixFromBinPath (argv0); + includeDirsSet = appendStrSet(dataDirsSet, NULL, INCLUDE_DIR_SUFFIX); - if (pbase == NULL) - { - /* A bad path. Skip. */ - } - else - { - if (_setPaths (pbase)) - { - /* Successfully set. */ - break; - } - else - { - /* Include and lib weren't where expected. */ - } - } - } - /* Case 3 */ - setMainValue ("includedir", SDCC_INCLUDE_DIR); - setMainValue ("libdir", SDCC_LIB_DIR); - } while (0); + if ((p = getenv(SDCC_INCLUDE_NAME)) != NULL) + addSetHead(&includeDirsSet, p); + + if (options.printSearchDirs) { + printf("includedir:\n"); + fputStrSet(stdout, includeDirsSet); + } +} + +/* Set system lib path */ +static void +setLibPath(void) +{ + char *p; + + /* + * Search logic: + * + * 1. - $SDCC_LIB + * 2. - $SDCC_HOME/PREFIX2DATA_DIR/LIB_DIR_SUFFIX/ + * 3. - path(argv[0])/BIN2DATA_DIR/LIB_DIR_SUFFIX/ + * 4. - DATADIR/LIB_DIR_SUFFIX/ (only on *nix) + */ + + libDirsSet = appendStrSet(dataDirsSet, NULL, LIB_DIR_SUFFIX); + + if ((p = getenv(SDCC_LIB_NAME)) != NULL) + addSetHead(&libDirsSet, p); + + if (options.printSearchDirs) { + printf("libdir:\n"); + fputStrSet(stdout, libDirsSet); + } +} + +/* Set data path */ +static void +setDataPaths(const char *argv0) +{ + char *p; + char buf[PATH_MAX]; + + /* + * Search logic: + * + * 1. - $SDCC_HOME/PREFIX2DATA_DIR + * 2. - path(argv[0])/BIN2DATA_DIR + * 3. - DATADIR (only on *nix) + */ + + if ((p = getenv(SDCC_DIR_NAME)) != NULL) { + SNPRINTF(buf, sizeof buf, "%s" PREFIX2DATA_DIR, p); + addSet(&dataDirsSet, Safe_strdup(buf)); + } + + if ((p = getBinPath(argv0)) != NULL) { + SNPRINTF(buf, sizeof buf, "%s" BIN2DATA_DIR, p); + addSet(&dataDirsSet, Safe_strdup(buf)); + } + +#ifdef _WIN32 + if (peekSet(dataDirsSet) == NULL) { + /* this should never happen... */ + wassertl(0, "Can't get binary path"); + } +#else + addSet(&dataDirsSet, Safe_strdup(DATADIR)); +#endif + + if (options.printSearchDirs) { + printf("datadir:\n"); + fputStrSet(stdout, dataDirsSet); + } + + setIncludePath(); + setLibPath(); } static void @@ -1867,9 +2033,6 @@ main (int argc, char **argv, char **envp) if (port->init) port->init (); - // Create a default exe search path from the path to the sdcc command - - setDefaultOptions (); #ifdef JAMIN_DS390 if (ds390_jammed) { @@ -1879,15 +2042,17 @@ main (int argc, char **argv, char **envp) #endif parseCmdLine (argc, argv); + initValues (); + setBinPaths(argv[0]); + setDataPaths(argv[0]); + /* if no input then printUsage & exit */ - if (!options.c1mode && !fullSrcFileName && !nrelFiles) - { - printUsage (); - exit (0); - } + if (!options.c1mode && !fullSrcFileName && peekSet(relFilesSet) == NULL) { + if (!options.printSearchDirs) + printUsage(); - initValues (); - _discoverPaths (argv[0]); + exit(0); + } /* initMem() is expensive, but initMem() must called before port->finaliseOptions (). @@ -1910,23 +2075,23 @@ main (int argc, char **argv, char **envp) printf ("sdcc: Generating code...\n"); yyparse (); + + pclose(yyin); + deleteSetItem(&pipeSet, yyin); + if (fatalError) { exit (1); } - if (TARGET_IS_PIC) { - /* TSD PIC port hack - if the PIC port option is enabled - and SDCC is used to generate PIC code, then we will - generate .asm files in gpasm's format instead of SDCC's - assembler's format - */ -#if !OPT_DISABLE_PIC - picglue (); -#endif - } - else { - glue (); - } + if (port->general.do_glue != NULL) + (*port->general.do_glue)(); + else + { + /* this shouldn't happen */ + assert(FALSE); + /* in case of NDEBUG */ + glue(); + } if (!options.c1mode && !noAssemble) { @@ -1937,14 +2102,14 @@ main (int argc, char **argv, char **envp) } closeDumpFiles(); - if (cdbFile) - fclose (cdbFile); + if (options.debug && debugFile) + debugFile->closeFile(); if (!options.cc_only && !fatalError && !noAssemble && !options.c1mode && - (fullSrcFileName || nrelFiles)) + (fullSrcFileName || peekSet(relFilesSet) != NULL)) { if (port->linker.do_link) port->linker.do_link ();