[llvm] [llvm-exegesis] Kill process that recieve a signal (PR #86069)

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 21 18:13:55 PDT 2024


================
@@ -342,7 +342,7 @@ class SubProcessFunctionExecutorImpl
       return make_error<Failure>("Failed to attach to the child process: " +
                                  Twine(strerror(errno)));
 
-    if (wait(NULL) == -1) {
+    if (waitpid(ParentOrChildPID, NULL, 0) == -1) {
----------------
boomanaiden154 wrote:

That's a good point. The suggested refactoring would definitely make the code cleaner. I'm going to land this PR and then open up another PR with the suggested refactoring to try and keep the history cleanish. Thanks for the suggestion!

https://github.com/llvm/llvm-project/pull/86069


More information about the llvm-commits mailing list