labath wrote:

My first thought when seeing (before opening the phabricator patch) this was 
"wouldn't this be better off as a separate API". After opening the link, I saw 
that this was the last comment on it as well. So, what has changed since then?

My think is that, our logging API is a fairly complex piece of machinery, and I 
think this is largely due to the fact that we're allowing the user to customize 
it (change the target, verbosity, etc.). For the this log channel, you 
basically take away all of that, so what's the point in going through all of 
that? Why not just have a single `SystemLog` function that does what you want?

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

Reply via email to