[PATCH] D62371: [llvm-symbolizer] Flush output on bad input

Igor Kudrin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 27 08:13:46 PDT 2019


ikudrin added inline comments.


================
Comment at: test/tools/llvm-symbolizer/Inputs/flush-output.py:8
+    process.kill()
+    sys.exit(1)
+
----------------
It looks like when `process.kill()` is executed, the main thread stops waiting, processes the rest and exits with the exit status `0`.

[[ https://docs.python.org/3/library/sys.html#sys.exit | The python documentation]] says:
> Since exit() ultimately “only” raises an exception, it will only exit the process when called from the main thread, and the exception is not intercepted.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62371/new/

https://reviews.llvm.org/D62371





More information about the llvm-commits mailing list