[profile] use GetComputerNameExW instead of gethostname on Windows

[inglorion]

Summary

In profile data paths, we replace "%h" with the hostname of the machine the program is running on. On Windows, we used gethostname() to obtain the hostname. This requires linking with ws2_32. With this change, we instead get the hostname from GetComputerNameExW(), which does not require ws2_32.

[inglorion]

This was suggested by @amccarth on D27086.

[rnk]

lib/profile/InstrProfilingPort.h

18

This code seems to go out of its way to not include windows.h, I'd like to preserve that. Can we sink the implementation of compiler_rt_gethostname to a .c file?

[inglorion]

lib/profile/InstrProfilingPort.h

18

Good point, will do.

[inglorion]

Moved the implementation of the compatibility function to a .c file, as requested by @rnk.

[amccarth]

I like where this is going.

lib/profile/InstrProfilingUtil.c

71 ↗

(On Diff #79477)

TCHAR?! Since we're calling WideCharToMultiByte, it seems we're assuming "Unicode" mode, so I'd write WCHAR and keep it explicit.

82 ↗

(On Diff #79477)

This doesn't terminate Name. There's a gotcha in this API that if you provide the length for the input string (BufferSize here), it'll process exactly that many "characters" and not terminate the output string.

See the description of cchWideChar in https://msdn.microsoft.com/en-us/library/windows/desktop/dd374130(v=vs.85).aspx

If you know that Buffer is terminated, then it's safest to use -1 for cchWideChar.

[inglorion]

addressed problems pointed out by @amccarth

[amccarth]

I'm happy. Thanks for fixing those warning messages, too.

[zturner]

lib/profile/InstrProfilingUtil.c

71 ↗

(On Diff #79477)

If we're assuming unicode mode, then let's call GetComputeNameExW.

[inglorion]

Changed to use GetComputerNameExW. Thanks for all the suggestions!

[inglorion]

clang-format

lib/profile/InstrProfilingFile.c

static int parseFilenamePattern(const char *FilenamePat,
   }
   /* Check the filename for "%p", which indicates a pid-substitution. */
   for (I = 0; FilenamePat[I]; ++I)
     if (FilenamePat[I] == '%') {
       if (FilenamePat[++I] == 'p') {
         if (!NumPids++) {
           if (snprintf(PidChars, MAX_PID_SIZE, "%d", getpid()) <= 0) {
             PROF_WARN(
-                "Unable to parse filename pattern %s. Using the default name.",
+                "Unable to get pid for filename pattern %s. Using the default name.",
                 FilenamePat);
             return -1;
           }
         }
       } else if (FilenamePat[I] == 'h') {
         if (!NumHosts++)
           if (COMPILER_RT_GETHOSTNAME(Hostname, COMPILER_RT_MAX_HOSTLEN)) {
             PROF_WARN(
-                "Unable to parse filename pattern %s. Using the default name.",
+                "Unable to get hostname for filename pattern %s. Using the default name.",
                 FilenamePat);
             return -1;
           }
       } else if (containsMergeSpecifier(FilenamePat, I)) {
         if (MergingEnabled) {
           PROF_WARN("%%m specifier can only be specified once in %s.\n",
                     FilenamePat);
           return -1;

lib/profile/InstrProfilingPort.h

 #ifndef PROFILE_INSTRPROFILING_PORT_H_
 #define PROFILE_INSTRPROFILING_PORT_H_
 
 #ifdef _MSC_VER
 #define COMPILER_RT_ALIGNAS(x) __declspec(align(x))
 #define COMPILER_RT_VISIBILITY
 /* FIXME: selectany does not have the same semantics as weak. */
 #define COMPILER_RT_WEAK __declspec(selectany)
 /* Need to include <windows.h> */
 #define COMPILER_RT_ALLOCA _alloca
 /* Need to include <stdio.h> and <io.h> */
 #define COMPILER_RT_FTRUNCATE(f,l) _chsize(_fileno(f),l)
 #elif __GNUC__
 #define COMPILER_RT_ALIGNAS(x) __attribute__((aligned(x)))
 #define COMPILER_RT_VISIBILITY __attribute__((visibility("hidden")))
 #define COMPILER_RT_WEAK __attribute__((weak))
 #define COMPILER_RT_ALLOCA __builtin_alloca

 #ifdef _MSC_VER
 #define COMPILER_RT_SECTION(Sect) __declspec(allocate(Sect))
 #else
 #define COMPILER_RT_SECTION(Sect) __attribute__((section(Sect)))
 #endif
 
 #define COMPILER_RT_MAX_HOSTLEN 128
 #ifdef _MSC_VER
-#define COMPILER_RT_GETHOSTNAME(Name, Len) gethostname(Name, Len)
+#include <windows.h>
+inline int compiler_rt_gethostname(LPSTR Name, int Len) {
+  TCHAR Buffer[256];
+  DWORD BufferSize = sizeof(Buffer);
+  BOOL Result = GetComputerNameEx(ComputerNameDnsFullyQualified, Buffer, &BufferSize);
+  if (!Result) return -1;
+  if (WideCharToMultiByte(CP_UTF8,
+        0,
+        Buffer,
+        BufferSize,
+        Name,
+        Len,
+        NULL,
+        NULL) == 0) return -1;
+  return 0;
+}
+#define COMPILER_RT_GETHOSTNAME(Name, Len) compiler_rt_gethostname(Name, Len)
 #elif defined(__ORBIS__)
 #define COMPILER_RT_GETHOSTNAME(Name, Len) ((void)(Name), (void)(Len), (-1))
 #else
 #define COMPILER_RT_GETHOSTNAME(Name, Len) lprofGetHostName(Name, Len)
 #define COMPILER_RT_HAS_UNAME 1
 #endif
 
 #if COMPILER_RT_HAS_ATOMICS == 1