| Issue |
147792
|
| Summary |
[lldb-dap] Revisit runInTerminal process communication
|
| Labels |
new issue
|
| Assignees |
|
| Reporter |
ashgti
|
When using `runInTerminal`, lldb-dap uses a fifo to communicate between the lldb-dap process and the launched process.
This communication is flaky on some platforms, for example on arm CI hosts we disable this test and I've seen this be unstable in some cases as well.
Additionally, we've got some implementation quirks like the fact we leak `std::future` objects when a timeout occurs:
https://github.com/llvm/llvm-project/blob/a63846b475bacfda49eb00016e0dc43c9ab1aa7d/lldb/tools/lldb-dap/FifoFiles.cpp#L62-L69
I think it might be worth revisiting this approach, for example we could use a socket instead of a fifo.
![]()
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs
