[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