| Issue |
147796
|
| Summary |
[lldb-dap] Revisit runInTerminal process communication
|
| Labels |
new issue
|
| Assignees |
|
| Reporter |
ashgti
|
Today, when we use the `runInTerminal` reverse request to open a file in a terminal we are using a FIFO to communicate between the processes.
This has been unstable in the CI and as a result we've disabled the tests for some architectures/platforms.
Additionally, we have issues where we are leaking `std::future` objects on timeout errors:
https://github.com/llvm/llvm-project/blob/a63846b475bacfda49eb00016e0dc43c9ab1aa7d/lldb/tools/lldb-dap/FifoFiles.cpp#L62-L69
It may be worth revisiting this and seeing if we could use a different approach, for example, we could use a socket instead of FIFO.
![]()
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs
