krytarowski added inline comments.
================ Comment at: include/lldb/Host/Config.h:35 + +#include "lldb/Host/openbsd/Config.h" ---------------- krytarowski wrote: > Missing in patch? I think that Config.h should go away. It's almost empty. But it's beyond the scope of this patch. While there I would sort this list of includes again. ================ Comment at: include/lldb/Host/HostInfo.h:55 +#elif defined(__OpenBSD__) +#include "lldb/Host/openbsd/HostInfoOpenBSD.h" +#define HOST_INFO_TYPE HostInfoOpenBSD ---------------- I would sort includes here. Repository: rL LLVM https://reviews.llvm.org/D31131 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
