labath accepted this revision.
labath added a comment.

Yes, this looks better, though I am mildly worried about the use of timeouts.

There probably isn't anything better we can do for the "launch" case, but we 
could definitely come up with something better for the "attach" case. I think 
we can leave the attach code as-is for now, as the symmetry with the launch 
code is nice, but this will need to be revisited if the test turns out to be 
flaky.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81200



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to