compiler.h changes for Windows MSVC support (#6623)

Summary:
Adding some MSVC (Windows) specific macros to compiler.h to make sure we can build with MSVC.



Differential Revision: D65328573

Author: tarun292

runtime/platform/compiler.h

@@ -100,22 +100,38 @@
 #endif // (__cplusplus) >= 202002L
 
 /// Define a C symbol with weak linkage.
+#ifdef _WIN32
+// There currently doesn't seem to be a great way to do this in Windows and given
+// that weak linkage is not really critical on Windows, we'll just leave it as a stub.
+#define ET_WEAK
+#else
 #define ET_WEAK __attribute__((weak))
+#endif
 
 /**
  * Annotation marking a function as printf-like, providing compiler support
  * for format string argument checking.
  */
+#ifdef _WIN32
+#include <sal.h>
+#define ET_PRINTFLIKE(_string_index, _va_index) _Printf_format_string_
+#else
 #define ET_PRINTFLIKE(_string_index, _va_index) \
   __attribute__((format(printf, _string_index, _va_index)))
-
-/// Name of the source file without a directory string.
-#define ET_SHORT_FILENAME (__builtin_strrchr("/" __FILE__, '/') + 1)
+#endif
 
 #ifndef __has_builtin
 #define __has_builtin(x) (0)
 #endif
 
+#if __has_builtin(__builtin_strrchr)
+/// Name of the source file without a directory string.
+#define ET_SHORT_FILENAME (__builtin_strrchr("/" __FILE__, '/') + 1)
+#else
+#include <cstring>
+#define ET_SHORT_FILENAME (strchr("/" __FILE__, '/') + 1)
+#endif
+
 #if __has_builtin(__builtin_LINE)
 /// Current line as an integer.
 #define ET_LINE __builtin_LINE()