From: Johann Glaser Date: Sat, 16 Mar 2013 21:04:55 +0000 (+0100) Subject: added error checking at execution of ABC X-Git-Tag: yosys-0.2.0~715^2~2 X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=0cb4a5936f92f0a526b4082a3cd52b3d98bae2fb;p=yosys.git added error checking at execution of ABC Signed-off-by: Clifford Wolf --- diff --git a/passes/abc/abc.cc b/passes/abc/abc.cc index c2c03e4a5..04d7ac67d 100644 --- a/passes/abc/abc.cc +++ b/passes/abc/abc.cc @@ -441,10 +441,28 @@ static void abc_module(RTLIL::Design *design, RTLIL::Module *current_module, std else snprintf(buffer, 1024, "%s -c 'read_verilog %s/input.v; read_library %s/stdcells.genlib; " "map; write_verilog %s/output.v' 2>&1", exe_file.c_str(), tempdir_name, tempdir_name, tempdir_name); + errno = ENOMEM; // popen does not set errno if memory allocation fails, therefore set it by hand f = popen(buffer, "r"); + if (!f) { + log("ABC: popen failed: %d, %s\n",errno,sys_errlist[errno]); + assert(0); + } while (fgets(buffer, 1024, f) != NULL) log("ABC: %s", buffer); - fclose(f); + errno = 0; + int ret = pclose(f); + if (ret < 0) { + log("ABC: pclose failed: %d, %s\n",errno,sys_errlist[errno]); + assert(0); + } + if (WEXITSTATUS(ret) != 0) { + switch (WEXITSTATUS(ret)) { + case 127: log("ABC: execution of command \"%s\" failed: Command not found\n",exe_file.c_str()); break; + case 126: log("ABC: execution of command \"%s\" failed: Command not executable\n",exe_file.c_str()); break; + default: log("ABC: execution of command \"%s\" failed: the shell returned %d\n",exe_file.c_str(),WEXITSTATUS(ret)); break; + } + assert(0); + } if (asprintf(&p, "%s/output.v", tempdir_name) < 0) abort(); f = fopen(p, "rt");