[PATCH] D138942: Support: Make Wait's SecondsToWait be Optional [NFC]

Serge Pavlov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 30 22:04:27 PST 2022


sepavloff added inline comments.


================
Comment at: llvm/lib/Support/Windows/Program.inc:427
   if (WaitStatus == WAIT_TIMEOUT) {
     if (SecondsToWait) {
       if (!TerminateProcess(PI.Process, 1)) {
----------------
You should put here `*SecondsToWait > 0`.


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

https://reviews.llvm.org/D138942



More information about the llvm-commits mailing list