Lgtm thanks<br><div class="gmail_quote"><div dir="ltr">On Tue, Dec 13, 2016 at 2:33 AM Hafiz Abid Qadeer via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">abidh created this revision.<br class="gmail_msg">
abidh added a reviewer: zturner.<br class="gmail_msg">
abidh added a subscriber: lldb-commits.<br class="gmail_msg">
<br class="gmail_msg">
This code was probably needed to support VS2013 and is not needed now. I have built it with VS and mingw. Ok to remove it?<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D27707" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D27707</a><br class="gmail_msg">
<br class="gmail_msg">
Files:<br class="gmail_msg">
include/lldb/Host/windows/PosixApi.h<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
Index: include/lldb/Host/windows/PosixApi.h<br class="gmail_msg">
===================================================================<br class="gmail_msg">
--- include/lldb/Host/windows/PosixApi.h<br class="gmail_msg">
+++ include/lldb/Host/windows/PosixApi.h<br class="gmail_msg">
@@ -75,15 +75,6 @@<br class="gmail_msg">
<br class="gmail_msg">
#endif // _MSC_VER<br class="gmail_msg">
<br class="gmail_msg">
-// MSVC 2015 and higher have timespec. Otherwise we need to define it<br class="gmail_msg">
-// ourselves.<br class="gmail_msg">
-#if !defined(_MSC_VER) || _MSC_VER < 1900<br class="gmail_msg">
-struct timespec {<br class="gmail_msg">
- time_t tv_sec;<br class="gmail_msg">
- long tv_nsec;<br class="gmail_msg">
-};<br class="gmail_msg">
-#endif<br class="gmail_msg">
-<br class="gmail_msg">
// Various useful posix functions that are not present in Windows. We provide<br class="gmail_msg">
// custom implementations.<br class="gmail_msg">
int vasprintf(char **ret, const char *fmt, va_list ap);<br class="gmail_msg">
@@ -113,20 +104,4 @@<br class="gmail_msg">
inline pid_t fork(void) { LLVM_BUILTIN_UNREACHABLE; }<br class="gmail_msg">
inline pid_t setsid(void) { LLVM_BUILTIN_UNREACHABLE; }<br class="gmail_msg">
<br class="gmail_msg">
-// vsnprintf and snprintf are provided in MSVC 2015 and higher.<br class="gmail_msg">
-#if _MSC_VER < 1900<br class="gmail_msg">
-namespace lldb_private {<br class="gmail_msg">
-int vsnprintf(char *buffer, size_t count, const char *format, va_list argptr);<br class="gmail_msg">
-}<br class="gmail_msg">
-<br class="gmail_msg">
-// inline to avoid linkage conflicts<br class="gmail_msg">
-int inline snprintf(char *buffer, size_t count, const char *format, ...) {<br class="gmail_msg">
- va_list argptr;<br class="gmail_msg">
- va_start(argptr, format);<br class="gmail_msg">
- int r = lldb_private::vsnprintf(buffer, count, format, argptr);<br class="gmail_msg">
- va_end(argptr);<br class="gmail_msg">
- return r;<br class="gmail_msg">
-}<br class="gmail_msg">
-#endif<br class="gmail_msg">
-<br class="gmail_msg">
#endif<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>