JDevlieghere added a comment. In D128321#3602129 <https://reviews.llvm.org/D128321#3602129>, @clayborg wrote:
> I think we should allow this class to always exist and not conditionally > compile it in or out. Then we add a new Host.h method to emit a log message > in "lldb/Host/Host.h" and allow each host OS to emit a message. Maybe there > is a default implementation that emits the message to stderr, and then we > override this for in HostInfoMacOSX.h? Then each OS can use their OS logging > API of choice. Yeah, I considered that, but that would make Utility depend on Host which would (re)introduce the circular dependency between the two. The alternative would be to still make the class available unconditionally, but use target ifdefs to have different implementations. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128321/new/ https://reviews.llvm.org/D128321 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
