ashgti wrote:

> * do we need regular file support

We don't strictly need this for anything I am aware of, I can remove it for 
now. A 'better' solution would be to stat the file and then watch the directory 
for changes and re-stat the file. Thats used in libuv (nodejs). I think 
libdispatch on Windows doesn't actually do anything for files, so you don't get 
notified if a file on disk changes.

> * do we need the pipe _thread_

In order to support both anonymous pipes, like stdin, and named pipes we would 
need the thread.

We could specialize this further and have the thread only for anonymous pipes 
and use a different strategy for named pipes, but the current solution works 
for both.


https://github.com/llvm/llvm-project/pull/145621
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to