mirror of https://github.com/llvm/torch-mlir
874fdb7e42
On my local machine, `unzip` didn't exist (producing a "command not found" error), but CMake ignored the error. Although the build did succeed (because it found a previously-built version of libtorch), it seems better to abort builds on such failures, so this patch checks the return code of all external process invocations. Along similar lines, this patch also updates the shell scripts in `build_tools` to extensively use double-quoting to prevent unintentional word splitting or globbing. Since some of the scripts execute `rm` while using shell variables, this patch also adds the preamble `set -u` to abort execution if an undefined variable is referenced, so that we reduce the chances of executing `rm -rf /` if the path expression happens to refer to an undefined variable. |
||
|---|---|---|
| .. | ||
| torch-mlir-lsp-server | ||
| torch-mlir-opt | ||
| CMakeLists.txt | ||
| torchscript_e2e_test.sh | ||