labath added a comment.

Note: I am certain that there will be things which will fail for these kinds of 
"processes" -- there's plenty of `!= LLDB_INVALID_PID` checks, but most of them 
seem to be on the launch/attach code paths which are not exercised here. I am 
fine with fixing these as they are discovered -- the bigger question here is 
whether this is a reasonable direction to take lldb in. I think it is, because 
most of the `!= LLDB_INVALID_PID` checks look like they could be replaced by a 
check of the process state.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70238



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

Reply via email to