From: Ben Langmuir <blangmuir@apple.com>
Date: Thu, 19 Feb 2015 04:03:57 +0000 (+0000)
Subject: Revert adding hostname to module hash
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=40e06d7fc7a7c9630cdfd2626322e4701500f7ab;p=clang

Revert adding hostname to module hash

I didn't realize how easily the hostname could change - for example just
changing wireless networks seems to prompt it in some cases.

Users can always set their own local module cache path to avoid this.

This reverts commits r228592, 228594, 228601 and 228613.
rdar://19287368

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@229815 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Frontend/CompilerInvocation.cpp b/lib/Frontend/CompilerInvocation.cpp
index 7413ae2601..edfee21399 100644
--- a/lib/Frontend/CompilerInvocation.cpp
+++ b/lib/Frontend/CompilerInvocation.cpp
@@ -39,9 +39,6 @@
 #include <memory>
 #include <sys/stat.h>
 #include <system_error>
-#if LLVM_ON_UNIX
-#include <unistd.h> // for gethostname()
-#endif
 using namespace clang;
 
 //===----------------------------------------------------------------------===//
@@ -2028,20 +2025,6 @@ std::string CompilerInvocation::getModuleHash() const {
     }
   }
 
-#if LLVM_ON_UNIX
-  // The LockFileManager cannot tell when processes from another host are
-  // running, so mangle the hostname in to the module hash to separate them.
-  char hostname[256];
-  hostname[0] = 0;
-  if (gethostname(hostname, 255) == 0) {
-    // Forcibly null-terminate the result, since POSIX doesn't require that
-    // truncation result in an error or that truncated names be null-terminated.
-    hostname[sizeof(hostname)-1] = 0;
-    code = hash_combine(code, StringRef(hostname));
-  }
-  // Ignore failures in gethostname() by not including the hostname in the hash.
-#endif
-
   return llvm::APInt(64, code).toString(36, /*Signed=*/false);
 }