Revert "Merge remote-tracking branch 'github/upstream-with-swift' into HEAD"

This reverts commit 362dfa5, reversing
changes made to daa6759.

Author: jkorous-apple

README.txt

@@ -8,3 +8,4 @@ Compiler-RT is open source software. You may freely distribute it under the
 terms of the license agreement found in LICENSE.txt.
 
 ================================
+

lib/profile/InstrProfilingFile.c

@@ -12,7 +12,6 @@
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
-#include <signal.h>
 #ifdef _MSC_VER
 /* For _alloca. */
 #include <malloc.h>
@@ -63,7 +62,6 @@ static const char *getPNSStr(ProfileNameSpecifier PNS) {
 }
 
 #define MAX_PID_SIZE 16
-#define MAX_SIGNAL_HANDLERS 16
 /* Data structure holding the result of parsed filename pattern. */
 typedef struct lprofFilename {
   /* File name string possibly with %p or %h specifiers. */
@@ -85,14 +83,11 @@ typedef struct lprofFilename {
    * 2 profile data files. %1m is equivalent to %m. Also %m specifier
    * can only appear once at the end of the name pattern. */
   unsigned MergePoolSize;
-  char ExitOnSignals[MAX_SIGNAL_HANDLERS];
-  unsigned NumExitSignals;
   ProfileNameSpecifier PNS;
 } lprofFilename;
 
 COMPILER_RT_WEAK lprofFilename lprofCurFilename = {0,   0, 0, 0, {0},
-                                                   {0}, 0, 0, 0, {0}, 0,
-                                                   PNS_unknown};
+                                                   {0}, 0, 0, 0, PNS_unknown};
 
 static int getCurFilenameLength();
 static const char *getCurFilename(char *FilenameBuf, int ForceUseBuf);
@@ -291,19 +286,6 @@ static void truncateCurrentFile(void) {
   fclose(File);
 }
 
-static void exitSignalHandler(int sig) {
-  (void)sig;
-  exit(0);
-}
-
-static void installExitSignalHandlers(void) {
-  unsigned I;
-  for (I = 0; I < lprofCurFilename.NumExitSignals; ++I) {
-    lprofInstallSignalHandler(lprofCurFilename.ExitOnSignals[I],
-                              exitSignalHandler);
-  }
-}
-
 static const char *DefaultProfileName = "default.profraw";
 static void resetFilenameToDefault(void) {
   if (lprofCurFilename.FilenamePat && lprofCurFilename.OwnsFilenamePat) {
@@ -314,25 +296,14 @@ static void resetFilenameToDefault(void) {
   lprofCurFilename.PNS = PNS_default;
 }
 
-static int isDigit(char C) { return C >= '0' && C <= '9'; }
-
-static int isNonZeroDigit(char C) { return C >= '1' && C <= '9'; }
-
 static int containsMergeSpecifier(const char *FilenamePat, int I) {
   return (FilenamePat[I] == 'm' ||
-          (isNonZeroDigit(FilenamePat[I]) &&
+          (FilenamePat[I] >= '1' && FilenamePat[I] <= '9' &&
            /* If FilenamePat[I] is not '\0', the next byte is guaranteed
             * to be in-bound as the string is null terminated. */
            FilenamePat[I + 1] == 'm'));
 }
 
-static int containsExitOnSignalSpecifier(const char *FilenamePat, int I) {
-  if (!isNonZeroDigit(FilenamePat[I]))
-    return 0;
-  return (FilenamePat[I + 1] == 'x') ||
-         (isDigit(FilenamePat[I + 1]) && FilenamePat[I + 2] == 'x');
-}
-
 /* Parses the pattern string \p FilenamePat and stores the result to
  * lprofcurFilename structure. */
 static int parseFilenamePattern(const char *FilenamePat,
@@ -341,7 +312,6 @@ static int parseFilenamePattern(const char *FilenamePat,
   char *PidChars = &lprofCurFilename.PidChars[0];
   char *Hostname = &lprofCurFilename.Hostname[0];
   int MergingEnabled = 0;
-  char SignalNo;
 
   /* Clean up cached prefix and filename.  */
   if (lprofCurFilename.ProfilePathPrefix)
@@ -394,22 +364,6 @@ static int parseFilenamePattern(const char *FilenamePat,
           lprofCurFilename.MergePoolSize = FilenamePat[I] - '0';
           I++; /* advance to 'm' */
         }
-      } else if (containsExitOnSignalSpecifier(FilenamePat, I)) {
-        if (lprofCurFilename.NumExitSignals == MAX_SIGNAL_HANDLERS) {
-          PROF_WARN("%%x specifier has been specified too many times in %s.\n",
-                    FilenamePat);
-          return -1;
-        }
-        /* Grab the signal number. */
-        SignalNo = FilenamePat[I] - '0';
-        I++; /* advance to either another digit, or 'x' */
-        if (FilenamePat[I] != 'x') {
-          SignalNo = (SignalNo * 10) + (FilenamePat[I] - '0');
-          I++; /* advance to 'x' */
-        }
-        lprofCurFilename.ExitOnSignals[lprofCurFilename.NumExitSignals] =
-            SignalNo;
-        ++lprofCurFilename.NumExitSignals;
       }
     }
 
@@ -453,7 +407,6 @@ static void parseAndSetFilename(const char *FilenamePat,
   }
 
   truncateCurrentFile();
-  installExitSignalHandlers();
 }
 
 /* Return buffer length that is required to store the current profile
@@ -462,24 +415,18 @@ static void parseAndSetFilename(const char *FilenamePat,
 #define SIGLEN 24
 static int getCurFilenameLength() {
   int Len;
-  unsigned I;
   if (!lprofCurFilename.FilenamePat || !lprofCurFilename.FilenamePat[0])
     return 0;
 
   if (!(lprofCurFilename.NumPids || lprofCurFilename.NumHosts ||
-        lprofCurFilename.MergePoolSize || lprofCurFilename.NumExitSignals))
+        lprofCurFilename.MergePoolSize))
     return strlen(lprofCurFilename.FilenamePat);
 
   Len = strlen(lprofCurFilename.FilenamePat) +
         lprofCurFilename.NumPids * (strlen(lprofCurFilename.PidChars) - 2) +
         lprofCurFilename.NumHosts * (strlen(lprofCurFilename.Hostname) - 2);
   if (lprofCurFilename.MergePoolSize)
     Len += SIGLEN;
-  for (I = 0; I < lprofCurFilename.NumExitSignals; ++I) {
-    Len -= 3; /* Drop the '%', signal number, and the 'x'. */
-    if (lprofCurFilename.ExitOnSignals[I] >= 10)
-      --Len; /* Drop the second digit of the signal number. */
-  }
   return Len;
 }
 
@@ -496,7 +443,7 @@ static const char *getCurFilename(char *FilenameBuf, int ForceUseBuf) {
     return 0;
 
   if (!(lprofCurFilename.NumPids || lprofCurFilename.NumHosts ||
-        lprofCurFilename.MergePoolSize || lprofCurFilename.NumExitSignals)) {
+        lprofCurFilename.MergePoolSize)) {
     if (!ForceUseBuf)
       return lprofCurFilename.FilenamePat;
 
@@ -529,9 +476,6 @@ static const char *getCurFilename(char *FilenameBuf, int ForceUseBuf) {
         J += S;
         if (FilenamePat[I] != 'm')
           I++;
-      } else if (containsExitOnSignalSpecifier(FilenamePat, I)) {
-        while (FilenamePat[I] != 'x')
-          ++I;
       }
       /* Drop any unknown substitutions. */
     } else

lib/profile/InstrProfilingUtil.c

@@ -23,7 +23,6 @@
 #include <sys/utsname.h>
 #endif
 
-#include <signal.h>
 #include <stdlib.h>
 #include <string.h>
 
@@ -271,24 +270,6 @@ COMPILER_RT_VISIBILITY const char *lprofFindLastDirSeparator(const char *Path) {
   return Sep;
 }
 
-COMPILER_RT_VISIBILITY void lprofInstallSignalHandler(int sig,
-                                                      void (*handler)(int)) {
-#ifdef _WIN32
-  void (*err)(int) = signal(sig, handler);
-  if (err == SIG_ERR)
-    PROF_WARN("Unable to install an exit signal handler for %d (errno = %d).\n",
-              sig, errno);
-#else
-  struct sigaction sigact;
-  memset(&sigact, 0, sizeof(sigact));
-  sigact.sa_handler = handler;
-  int err = sigaction(sig, &sigact, NULL);
-  if (err)
-    PROF_WARN("Unable to install an exit signal handler for %d (errno = %d).\n",
-              sig, err);
-#endif
-}
-
 COMPILER_RT_VISIBILITY int lprofSuspendSigKill() {
 #if defined(__linux__)
   int PDeachSig = 0;

lib/profile/InstrProfilingUtil.h

@@ -59,8 +59,6 @@ int lprofGetHostName(char *Name, int Len);
 unsigned lprofBoolCmpXchg(void **Ptr, void *OldV, void *NewV);
 void *lprofPtrFetchAdd(void **Mem, long ByteIncr);
 
-void lprofInstallSignalHandler(int sig, void(*handler)(int));
-
 /* Temporarily suspend SIGKILL. Return value of 1 means a restore is needed.
  * Other return values mean no restore is needed.
  */

lib/tsan/rtl/tsan_rtl_thread.cc

@@ -275,14 +275,10 @@ void ThreadStart(ThreadState *thr, int tid, tid_t os_id, bool workerthread) {
 void ThreadFinish(ThreadState *thr) {
   ThreadCheckIgnore(thr);
   StatInc(thr, StatThreadFinish);
-  if (thr->stk_addr && thr->stk_size) {
-    MemoryResetRange(thr, /*pc=*/ 1, thr->stk_addr, thr->stk_size);
+  if (thr->stk_addr && thr->stk_size)
     DontNeedShadowFor(thr->stk_addr, thr->stk_size);
-  }
-  if (thr->tls_addr && thr->tls_size) {
-    MemoryResetRange(thr, /*pc=*/ 1, thr->tls_addr, thr->tls_size);
+  if (thr->tls_addr && thr->tls_size)
     DontNeedShadowFor(thr->tls_addr, thr->tls_size);
-  }
   thr->is_dead = true;
   ctx->thread_registry->FinishThread(thr->tid);
 }

test/lit.common.cfg

@@ -101,9 +101,6 @@ if re.match(r'^x86_64.*-linux', config.target_triple):
 if config.have_zlib == "1":
   config.available_features.add("zlib")
 
-if lit.util.isMacOSTriple(config.target_triple):
-   config.available_features.add('darwin')
-
 # Use ugly construction to explicitly prohibit "clang", "clang++" etc.
 # in RUN lines.
 config.substitutions.append(

test/lsan/lit.common.cfg

@@ -78,8 +78,3 @@ if re.search('mthumb', config.target_cflags) is not None:
   config.unsupported = True
 
 config.suffixes = ['.c', '.cc', '.cpp', '.mm']
-
-# Apple-Clang: Disable LSan
-if config.host_os == 'Darwin':
-  lit_config.note('Disabling LSan tests on Darwin')
-  config.unsupported = True

test/profile/instrprof-exit-on-signal.c

@@ -70,9 +70,3 @@ config.suffixes = ['.c', '.cc', '.cpp']
 
 if config.host_os not in ['Linux', 'Darwin', 'NetBSD', 'FreeBSD']:
   config.unsupported = True
-
-# Apple-internal: Disable LSan sanitizer_common tests
-# because AppleClang doesn't support LSan (rdar://problem/47381908).
-if config.tool_name == 'lsan' and config.host_os == 'Darwin':
-  lit_config.note('LSan sanitizer_common tests disabled')
-  config.unsupported = True