[Lldb-commits] [lldb] [LLDB][Windows]: Don't pass duplicate HANDLEs to CreateProcess (PR #165281)

[Charles Zablit via lldb-commits]

charles-zablit wrote:

> Perhaps the HANDLE array should be kept alive until the CreateProcess call?

That's worth ensuring as well. Given the non deterministic nature of the test failures, my bet is on the insertion order. In the original code, replacing `std::vector` with `llvm::UniqueVector` would give us both the insertion order and the proper lifetime.

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