X-Git-Url: http://nsz.repo.hu/git/?a=blobdiff_plain;f=main.c;h=6e55b36d053e36ad99d92c40ebd4840a29683e7f;hb=3cdead30b5c1d131d5c363188c26255514ea2fc8;hp=b3a90651030b4c8e83c0ddb35eb422548957be4d;hpb=708d819aa335a9b2c6d83b6a2c303a77b9a94e34;p=cparser diff --git a/main.c b/main.c index b3a9065..6e55b36 100644 --- a/main.c +++ b/main.c @@ -72,14 +72,15 @@ #include "driver/firm_opt.h" #include "driver/firm_cmdline.h" #include "adt/error.h" -#include "write_fluffy.h" -#include "write_caml.h" +#include "wrappergen/write_fluffy.h" +#include "wrappergen/write_caml.h" +#include "wrappergen/write_jna.h" #include "revision.h" #include "warning.h" #include "mangle.h" #ifndef PREPROCESSOR -#ifdef __APPLE__ +#ifndef __WIN32__ #define PREPROCESSOR "gcc -E -std=c99 -m32 -U__STRICT_ANSI__" #else #define PREPROCESSOR "cpp -std=c99 -m32 -U__STRICT_ANSI__" @@ -180,30 +181,24 @@ static void initialize_firm(void) static void get_output_name(char *buf, size_t buflen, const char *inputname, const char *newext) { - size_t last_dot = 0xffffffff; - size_t i = 0; - - if(inputname == NULL) { - snprintf(buf, buflen, "a%s", newext); - return; - } - - for(const char *c = inputname; *c != 0; ++c) { - if(*c == '.') - last_dot = i; - ++i; - } - if(last_dot == 0xffffffff) - last_dot = i; - - if(last_dot >= buflen) - panic("filename too long"); - memcpy(buf, inputname, last_dot); - - size_t extlen = strlen(newext) + 1; - if(extlen + last_dot >= buflen) + if (inputname == NULL) + inputname = "a"; + + char const *const last_slash = strrchr(inputname, '/'); + char const *const filename = + last_slash != NULL ? last_slash + 1 : inputname; + char const *const last_dot = strrchr(filename, '.'); + char const *const name_end = + last_dot != NULL ? last_dot : strchr(filename, '\0'); + + int const len = snprintf(buf, buflen, "%.*s%s", + (int)(name_end - filename), filename, newext); +#ifdef _WIN32 + if (len < 0 || buflen <= (size_t)len) +#else + if (buflen <= (size_t)len) +#endif panic("filename too long"); - memcpy(buf+last_dot, newext, extlen); } #include "builtins.h" @@ -231,42 +226,55 @@ static void lextest(FILE *in, const char *fname) do { lexer_next_preprocessing_token(); print_token(stdout, &lexer_token); - puts(""); - } while(lexer_token.type != T_EOF); + putchar('\n'); + } while (lexer_token.type != T_EOF); } static void add_flag(struct obstack *obst, const char *format, ...) { + char buf[4096]; va_list ap; - va_start(ap, format); - char buf[4096]; - vsnprintf(buf, sizeof(buf), format, ap); + va_start(ap, format); +#ifdef _WIN32 + int len = +#endif + vsnprintf(buf, sizeof(buf), format, ap); + va_end(ap); - /* escape stuff... */ obstack_1grow(obst, ' '); +#ifdef _WIN32 + obstack_1grow(obst, '"'); + obstack_grow(obst, buf, len); + obstack_1grow(obst, '"'); +#else + /* escape stuff... */ for (char *c = buf; *c != '\0'; ++c) { switch(*c) { - case '"': - case '\'': - case '`': case ' ': - case '\t': - case '\n': - case '\r': - case '\\': + case '"': case '$': + case '&': case '(': case ')': + case ';': + case '<': + case '>': + case '\'': + case '\\': + case '\n': + case '\r': + case '\t': + case '`': + case '|': obstack_1grow(obst, '\\'); - /* fallthrough */ + /* FALLTHROUGH */ default: obstack_1grow(obst, *c); break; } } - - va_end(ap); +#endif } static const char *type_to_string(type_t *type) @@ -284,33 +292,44 @@ static FILE *preprocess(const char *fname) /* setup default defines */ add_flag(&cppflags_obst, "-U__WCHAR_TYPE__"); - add_flag(&cppflags_obst, "-D__WCHAR_TYPE__=%s", - type_to_string(type_wchar_t)); + add_flag(&cppflags_obst, "-D__WCHAR_TYPE__=%s", type_to_string(type_wchar_t)); add_flag(&cppflags_obst, "-U__SIZE_TYPE__"); add_flag(&cppflags_obst, "-D__SIZE_TYPE__=%s", type_to_string(type_size_t)); + /* TODO hack... */ + add_flag(&cppflags_obst, "-D__builtin_abort=abort"); + add_flag(&cppflags_obst, "-D__builtin_abs=abs"); + add_flag(&cppflags_obst, "-D__builtin_exit=exit"); + add_flag(&cppflags_obst, "-D__builtin_malloc=malloc"); + add_flag(&cppflags_obst, "-D__builtin_memcmp=memcmp"); + add_flag(&cppflags_obst, "-D__builtin_memcpy=memcpy"); + add_flag(&cppflags_obst, "-D__builtin_memset=memset"); + add_flag(&cppflags_obst, "-D__builtin_strlen=strlen"); + add_flag(&cppflags_obst, "-D__builtin_strcmp=strcmp"); + add_flag(&cppflags_obst, "-D__builtin_strcpy=strcpy"); + /* handle dependency generation */ if (dep_target[0] != '\0') { add_flag(&cppflags_obst, "-MF"); - add_flag(&cppflags_obst, "%s", dep_target); + add_flag(&cppflags_obst, dep_target); if (outname != NULL) { add_flag(&cppflags_obst, "-MQ"); - add_flag(&cppflags_obst, "%s", outname); + add_flag(&cppflags_obst, outname); } } if (flags[0] != '\0') { obstack_printf(&cppflags_obst, " %s", flags); } - add_flag(&cppflags_obst, "%s", fname); + add_flag(&cppflags_obst, fname); obstack_1grow(&cppflags_obst, '\0'); const char *buf = obstack_finish(&cppflags_obst); - if(verbose) { + if (verbose) { puts(buf); } FILE *f = popen(buf, "r"); - if(f == NULL) { + if (f == NULL) { fprintf(stderr, "invoking preprocessor failed\n"); exit(1); } @@ -323,22 +342,46 @@ static void assemble(const char *out, const char *in) char buf[4096]; snprintf(buf, sizeof(buf), "%s %s -o %s", ASSEMBLER, in, out); - if(verbose) { + if (verbose) { puts(buf); } int err = system(buf); - if(err != 0) { + if (err != 0) { fprintf(stderr, "assembler reported an error\n"); exit(1); } } +static void print_file_name(const char *file) +{ + add_flag(&ldflags_obst, "-print-file-name=%s", file); + + obstack_1grow(&ldflags_obst, '\0'); + const char *flags = obstack_finish(&ldflags_obst); + + /* construct commandline */ + obstack_printf(&ldflags_obst, "%s ", LINKER); + obstack_printf(&ldflags_obst, "%s", flags); + obstack_1grow(&ldflags_obst, '\0'); + + char *commandline = obstack_finish(&ldflags_obst); + + if (verbose) { + puts(commandline); + } + int err = system(commandline); + if (err != EXIT_SUCCESS) { + fprintf(stderr, "linker reported an error\n"); + exit(1); + } +} + static const char *try_dir(const char *dir) { - if(dir == NULL) + if (dir == NULL) return dir; - if(access(dir, R_OK | W_OK | X_OK) == 0) + if (access(dir, R_OK | W_OK | X_OK) == 0) return dir; return NULL; } @@ -347,29 +390,29 @@ static const char *get_tempdir(void) { static const char *tmpdir = NULL; - if(tmpdir != NULL) + if (tmpdir != NULL) return tmpdir; - if(tmpdir == NULL) + if (tmpdir == NULL) tmpdir = try_dir(getenv("TMPDIR")); - if(tmpdir == NULL) + if (tmpdir == NULL) tmpdir = try_dir(getenv("TMP")); - if(tmpdir == NULL) + if (tmpdir == NULL) tmpdir = try_dir(getenv("TEMP")); #ifdef P_tmpdir - if(tmpdir == NULL) + if (tmpdir == NULL) tmpdir = try_dir(P_tmpdir); #endif - if(tmpdir == NULL) + if (tmpdir == NULL) tmpdir = try_dir("/var/tmp"); - if(tmpdir == NULL) + if (tmpdir == NULL) tmpdir = try_dir("/usr/tmp"); - if(tmpdir == NULL) + if (tmpdir == NULL) tmpdir = try_dir("/tmp"); - if(tmpdir == NULL) + if (tmpdir == NULL) tmpdir = "."; return tmpdir; @@ -395,13 +438,13 @@ static FILE *make_temp_file(char *buffer, size_t buflen, const char *prefix) snprintf(buffer, buflen, "%s/%sXXXXXX", tempdir, prefix); int fd = mkstemp(buffer); - if(fd == -1) { + if (fd == -1) { fprintf(stderr, "couldn't create temporary file: %s\n", strerror(errno)); exit(1); } FILE *out = fdopen(fd, "w"); - if(out == NULL) { + if (out == NULL) { fprintf(stderr, "couldn't create temporary file FILE*\n"); exit(1); } @@ -460,7 +503,8 @@ typedef enum compile_mode_t { LexTest, PrintAst, PrintFluffy, - PrintCaml + PrintCaml, + PrintJna } compile_mode_t; static void usage(const char *argv0) @@ -468,19 +512,22 @@ static void usage(const char *argv0) fprintf(stderr, "Usage %s input [-o output] [-c]\n", argv0); } -static void print_cparser_version(void) { - firm_version_t ver; - firm_get_version(&ver); - +static void print_cparser_version(void) +{ printf("cparser (%s) using libFirm (%u.%u", - cparser_REVISION, ver.major, ver.minor); - if(ver.revision[0] != 0) { + cparser_REVISION, ir_get_version_major(), + ir_get_version_minor()); + + const char *revision = ir_get_version_revision(); + if (revision[0] != 0) { putchar(' '); - fputs(ver.revision, stdout); + fputs(revision, stdout); } - if(ver.build[0] != 0) { + + const char *build = ir_get_version_build(); + if (build[0] != 0) { putchar(' '); - fputs(ver.build, stdout); + fputs(build, stdout); } puts(")\n"); } @@ -505,7 +552,7 @@ static void copy_file(FILE *dest, FILE *input) while (!feof(input) && !ferror(dest)) { size_t read = fread(buf, 1, sizeof(buf), input); - if(fwrite(buf, 1, read, dest) != read) { + if (fwrite(buf, 1, read, dest) != read) { perror("couldn't write output"); } } @@ -532,7 +579,7 @@ static FILE *open_file(const char *filename) } FILE *in = fopen(filename, "r"); - if(in == NULL) { + if (in == NULL) { fprintf(stderr, "Couldn't open '%s': %s\n", filename, strerror(errno)); exit(1); @@ -593,6 +640,7 @@ int main(int argc, char **argv) initialize_firm(); const char *dumpfunction = NULL; + const char *print_file_name_file = NULL; compile_mode_t mode = CompileAssembleLink; int opt_level = 1; int result = EXIT_SUCCESS; @@ -616,15 +664,15 @@ int main(int argc, char **argv) #define GET_ARG_AFTER(def, args) \ def = &arg[sizeof(args)-1]; \ - if(def[0] == '\0') { \ + if (def[0] == '\0') { \ ++i; \ - if(i >= argc) { \ + if (i >= argc) { \ fprintf(stderr, "error: expected argument after '" args "'\n"); \ argument_errors = true; \ break; \ } \ def = argv[i]; \ - if(def[0] == '-' && def[1] != '\0') { \ + if (def[0] == '-' && def[1] != '\0') { \ fprintf(stderr, "error: expected argument after '" args "'\n"); \ argument_errors = true; \ continue; \ @@ -636,7 +684,7 @@ int main(int argc, char **argv) /* early options parsing (find out optimisation level and OS) */ for (int i = 1; i < argc; ++i) { const char *arg = argv[i]; - if(arg[0] != '-') + if (arg[0] != '-') continue; const char *option = &arg[1]; @@ -686,50 +734,50 @@ int main(int argc, char **argv) filetype_t forced_filetype = FILETYPE_AUTODETECT; bool help_displayed = false; bool argument_errors = false; - for(int i = 1; i < argc; ++i) { + for (int i = 1; i < argc; ++i) { const char *arg = argv[i]; if (arg[0] == '-' && arg[1] != '\0') { /* an option */ const char *option = &arg[1]; - if(option[0] == 'o') { + if (option[0] == 'o') { GET_ARG_AFTER(outname, "-o"); - } else if(option[0] == 'g') { + } else if (option[0] == 'g') { set_be_option("debuginfo=stabs"); set_be_option("omitfp=no"); set_be_option("ia32-nooptcc=yes"); - } else if(SINGLE_OPTION('c')) { + } else if (SINGLE_OPTION('c')) { mode = CompileAssemble; - } else if(SINGLE_OPTION('E')) { + } else if (SINGLE_OPTION('E')) { mode = PreprocessOnly; - } else if(SINGLE_OPTION('S')) { + } else if (SINGLE_OPTION('S')) { mode = Compile; - } else if(option[0] == 'O') { + } else if (option[0] == 'O') { continue; - } else if(option[0] == 'I') { + } else if (option[0] == 'I') { const char *opt; GET_ARG_AFTER(opt, "-I"); add_flag(&cppflags_obst, "-I%s", opt); - } else if(option[0] == 'D') { + } else if (option[0] == 'D') { const char *opt; GET_ARG_AFTER(opt, "-D"); add_flag(&cppflags_obst, "-D%s", opt); - } else if(option[0] == 'U') { + } else if (option[0] == 'U') { const char *opt; GET_ARG_AFTER(opt, "-U"); add_flag(&cppflags_obst, "-U%s", opt); - } else if(option[0] == 'l') { + } else if (option[0] == 'l') { const char *opt; GET_ARG_AFTER(opt, "-l"); add_flag(&ldflags_obst, "-l%s", opt); - } else if(option[0] == 'L') { + } else if (option[0] == 'L') { const char *opt; GET_ARG_AFTER(opt, "-L"); add_flag(&ldflags_obst, "-L%s", opt); - } else if(SINGLE_OPTION('v')) { + } else if (SINGLE_OPTION('v')) { verbose = 1; - } else if(SINGLE_OPTION('w')) { + } else if (SINGLE_OPTION('w')) { memset(&warning, 0, sizeof(warning)); - } else if(option[0] == 'x') { + } else if (option[0] == 'x') { const char *opt; GET_ARG_AFTER(opt, "-x"); forced_filetype = get_filetype_from_string(opt); @@ -744,7 +792,7 @@ int main(int argc, char **argv) streq(option, "MD")) { construct_dep_target = true; add_flag(&cppflags_obst, "-%s", option); - } else if(streq(option, "MM") || + } else if (streq(option, "MM") || streq(option, "MP")) { add_flag(&cppflags_obst, "-%s", option); } else if (streq(option, "MT") || @@ -754,6 +802,20 @@ int main(int argc, char **argv) GET_ARG_AFTER(opt, "-MT"); add_flag(&cppflags_obst, "-%s", option); add_flag(&cppflags_obst, "%s", opt); + } else if (streq(option, "include")) { + const char *opt; + GET_ARG_AFTER(opt, "-include"); + add_flag(&cppflags_obst, "-include"); + add_flag(&cppflags_obst, "%s", opt); + } else if (streq(option, "isystem")) { + const char *opt; + GET_ARG_AFTER(opt, "-isystem"); + add_flag(&cppflags_obst, "-isystem"); + add_flag(&cppflags_obst, "%s", opt); + } else if (streq(option, "nostdinc") + || streq(option, "trigraphs")) { + /* pass these through to the preprocessor */ + add_flag(&cppflags_obst, "%s", arg); } else if (streq(option, "pipe")) { /* here for gcc compatibility */ } else if (option[0] == 'f') { @@ -764,6 +826,11 @@ int main(int argc, char **argv) strstart(orig_opt, "align-jumps=") || strstart(orig_opt, "align-functions=")) { fprintf(stderr, "ignoring gcc option '-f%s'\n", orig_opt); + } else if (strstart(orig_opt, "input-charset=")) { + char const* const encoding = strchr(orig_opt, '=') + 1; + select_input_encoding(encoding); + } else if (streq(orig_opt, "verbose-asm")) { + /* ignore: we always print verbose assembler */ } else { char const *opt = orig_opt; bool truth_value = true; @@ -772,19 +839,23 @@ int main(int argc, char **argv) opt += 3; } - if (streq(opt, "dollars-in-identifiers")) { - allow_dollar_in_symbol = truth_value; - } if (streq(opt, "builtins")) { + if (streq(opt, "builtins")) { use_builtins = truth_value; + } else if (streq(opt, "dollars-in-identifiers")) { + allow_dollar_in_symbol = truth_value; + } else if (streq(opt, "omit-frame-pointer")) { + set_be_option(truth_value ? "omitfp" : "omitfp=no"); } else if (streq(opt, "short-wchar")) { wchar_atomic_kind = truth_value ? ATOMIC_TYPE_USHORT : ATOMIC_TYPE_INT; - } else if (streq(opt, "syntax-only")) { - mode = truth_value ? ParseOnly : CompileAssembleLink; - } else if (streq(opt, "omit-frame-pointer")) { - set_be_option(truth_value ? "omitfp" : "omitfp=no"); + } else if (streq(opt, "signed-char")) { + char_is_signed = truth_value; } else if (streq(opt, "strength-reduce")) { firm_option(truth_value ? "strength-red" : "no-strength-red"); + } else if (streq(opt, "syntax-only")) { + mode = truth_value ? ParseOnly : CompileAssembleLink; + } else if (streq(opt, "unsigned-char")) { + char_is_signed = !truth_value; } else if (streq(opt, "fast-math") || streq(opt, "jump-tables") || streq(opt, "unroll-loops") || @@ -821,14 +892,23 @@ int main(int argc, char **argv) strncpy(cpu_arch, opt, sizeof(cpu_arch)); } } else if (option[0] == 'W') { - if (strstart(option + 1, "l,")) // a gcc-style linker option - { + if (strstart(option + 1, "p,")) { + // pass options directly to the preprocessor + const char *opt; + GET_ARG_AFTER(opt, "-Wp,"); + add_flag(&cppflags_obst, "-Wp,%s", opt); + } else if (strstart(option + 1, "l,")) { + // pass options directly to the linker const char *opt; GET_ARG_AFTER(opt, "-Wl,"); add_flag(&ldflags_obst, "-Wl,%s", opt); + } else if (streq(option + 1, "no-trigraphs") + || streq(option + 1, "undef")) { + add_flag(&cppflags_obst, "%s", arg); + } else { + set_warning_opt(&option[1]); } - else set_warning_opt(&option[1]); - } else if(option[0] == 'm') { + } else if (option[0] == 'm') { /* -m options */ const char *opt; char arch_opt[64]; @@ -868,7 +948,7 @@ int main(int argc, char **argv) fprintf(stderr, "error: option -mfpumath supports only 387 or sse\n"); argument_errors = true; } - if(!argument_errors) { + if (!argument_errors) { snprintf(arch_opt, sizeof(arch_opt), "%s-fpunit=%s", cpu_arch, opt); int res = firm_be_option(arch_opt); if (res == 0) @@ -925,6 +1005,8 @@ int main(int argc, char **argv) (fprintf(stderr, "warning: ignoring gcc option '%s'\n", arg), standard); } else if (streq(option, "version")) { print_cparser_version(); + } else if (strstart(option, "print-file-name=")) { + GET_ARG_AFTER(print_file_name_file, "-print-file-name="); } else if (option[0] == '-') { /* double dash option */ ++option; @@ -940,10 +1022,6 @@ int main(int argc, char **argv) } else if (streq(option, "no-ms")) { features_on &= ~_MS; features_off |= _MS; - } else if (streq(option, "signed-chars")) { - char_is_signed = true; - } else if (streq(option, "unsigned-chars")) { - char_is_signed = false; } else if (streq(option, "strict")) { strict_mode = true; } else if (streq(option, "lextest")) { @@ -960,12 +1038,14 @@ int main(int argc, char **argv) mode = PrintFluffy; } else if (streq(option, "print-caml")) { mode = PrintCaml; + } else if (streq(option, "print-jna")) { + mode = PrintJna; } else if (streq(option, "version")) { print_cparser_version(); exit(EXIT_SUCCESS); } else if (streq(option, "dump-function")) { ++i; - if(i >= argc) { + if (i >= argc) { fprintf(stderr, "error: " "expected argument after '--dump-function'\n"); argument_errors = true; @@ -1028,6 +1108,11 @@ int main(int argc, char **argv) } } + if (print_file_name_file != NULL) { + print_file_name(print_file_name_file); + return 0; + } + if (files == NULL) { fprintf(stderr, "error: no input files specified\n"); argument_errors = true; @@ -1044,6 +1129,10 @@ int main(int argc, char **argv) /* we do the lowering in ast2firm */ firm_opt.lower_bitfields = FALSE; + /* set the c_mode here, types depends on it */ + c_mode |= features_on; + c_mode &= ~features_off; + gen_firm_init(); init_symbol_table(); init_types(); @@ -1074,6 +1163,7 @@ int main(int argc, char **argv) case PrintAst: case PrintFluffy: case PrintCaml: + case PrintJna: case LexTest: case PreprocessOnly: case ParseOnly: @@ -1109,7 +1199,7 @@ int main(int argc, char **argv) out = stdout; } else { out = fopen(outname, "w"); - if(out == NULL) { + if (out == NULL) { fprintf(stderr, "Couldn't open '%s' for writing: %s\n", outname, strerror(errno)); return 1; @@ -1136,9 +1226,17 @@ int main(int argc, char **argv) FILE *preprocessed_in = NULL; switch (filetype) { - case FILETYPE_C: filetype = FILETYPE_PREPROCESSED_C; goto preprocess; - case FILETYPE_CXX: filetype = FILETYPE_PREPROCESSED_CXX; goto preprocess; - case FILETYPE_ASSEMBLER: filetype = FILETYPE_PREPROCESSED_ASSEMBLER; goto preprocess; + case FILETYPE_C: + filetype = FILETYPE_PREPROCESSED_C; + goto preprocess; + case FILETYPE_CXX: + filetype = FILETYPE_PREPROCESSED_CXX; + goto preprocess; + case FILETYPE_ASSEMBLER: + filetype = FILETYPE_PREPROCESSED_ASSEMBLER; + add_flag(&cppflags_obst, "-x"); + add_flag(&cppflags_obst, "assembler-with-cpp"); + goto preprocess; preprocess: /* no support for input on FILE* yet */ if (in != NULL) @@ -1149,6 +1247,10 @@ preprocess: copy_file(out, preprocessed_in); int result = pclose(preprocessed_in); fclose(out); + /* remove output file in case of error */ + if (out != stdout && result != EXIT_SUCCESS) { + unlink(outname); + } return result; } @@ -1160,7 +1262,7 @@ preprocess: } FILE *asm_out; - if(mode == Compile) { + if (mode == Compile) { asm_out = out; } else { asm_out = make_temp_file(asm_tempfile, sizeof(asm_tempfile), "ccs"); @@ -1215,6 +1317,7 @@ default_cxx_warn: do_parsing: c_mode |= features_on; c_mode &= ~features_off; + init_tokens(); translation_unit_t *const unit = do_parsing(in, filename); @@ -1225,31 +1328,37 @@ do_parsing: print_ast(unit); } - if(error_count > 0) { + if (error_count > 0) { /* parsing failed because of errors */ fprintf(stderr, "%u error(s), %u warning(s)\n", error_count, warning_count); result = EXIT_FAILURE; continue; - } else if(warning_count > 0) { + } else if (warning_count > 0) { fprintf(stderr, "%u warning(s)\n", warning_count); } if (in == preprocessed_in) { int pp_result = pclose(preprocessed_in); if (pp_result != EXIT_SUCCESS) { + /* remove output file */ + if (out != stdout) + unlink(outname); exit(EXIT_FAILURE); } } - if(mode == BenchmarkParser) { + if (mode == BenchmarkParser) { return result; - } else if(mode == PrintFluffy) { + } else if (mode == PrintFluffy) { write_fluffy_decls(out, unit); continue; } else if (mode == PrintCaml) { write_caml_decls(out, unit); continue; + } else if (mode == PrintJna) { + write_jna_decls(out, unit); + continue; } translation_unit_to_firm(unit); @@ -1263,16 +1372,16 @@ do_parsing: ident *id = new_id_from_str(dumpfunction); ir_graph *irg = NULL; int n_irgs = get_irp_n_irgs(); - for(int i = 0; i < n_irgs; ++i) { + for (int i = 0; i < n_irgs; ++i) { ir_graph *tirg = get_irp_irg(i); ident *irg_id = get_entity_ident(get_irg_entity(tirg)); - if(irg_id == id) { + if (irg_id == id) { irg = tirg; break; } } - if(irg == NULL) { + if (irg == NULL) { fprintf(stderr, "No graph for function '%s' found\n", dumpfunction); exit(1); @@ -1293,10 +1402,13 @@ do_parsing: if (in == preprocessed_in) { int pp_result = pclose(preprocessed_in); if (pp_result != EXIT_SUCCESS) { + /* remove output in error case */ + if (out != stdout) + unlink(outname); return pp_result; } } - if(asm_out != out) { + if (asm_out != out) { fclose(asm_out); } } @@ -1312,7 +1424,7 @@ do_parsing: if (filetype == FILETYPE_PREPROCESSED_ASSEMBLER) { char temp[1024]; const char *filename_o; - if(mode == CompileAssemble) { + if (mode == CompileAssemble) { fclose(out); filename_o = outname; } else { @@ -1333,11 +1445,14 @@ do_parsing: file->type = filetype; } - if (result != EXIT_SUCCESS) + if (result != EXIT_SUCCESS) { + if (out != stdout) + unlink(outname); return result; + } /* link program file */ - if(mode == CompileAssembleLink) { + if (mode == CompileAssembleLink) { obstack_1grow(&ldflags_obst, '\0'); const char *flags = obstack_finish(&ldflags_obst); @@ -1358,11 +1473,11 @@ do_parsing: char *commandline = obstack_finish(&file_obst); - if(verbose) { + if (verbose) { puts(commandline); } int err = system(commandline); - if(err != EXIT_SUCCESS) { + if (err != EXIT_SUCCESS) { fprintf(stderr, "linker reported an error\n"); exit(1); }