diff options
| author | Xinliang David Li <davidxl@google.com> | 2017-08-15 03:13:01 +0000 |
|---|---|---|
| committer | Xinliang David Li <davidxl@google.com> | 2017-08-15 03:13:01 +0000 |
| commit | 1c9320cd4d970ebbb2830d5c73a5120a53ffbd5c (patch) | |
| tree | 873570ec2024918760766ce6c6df3bd3df0d0293 /compiler-rt | |
| parent | 7a3da86823fb4bea6c80f9d7f972b3dc944acf19 (diff) | |
| download | bcm5719-llvm-1c9320cd4d970ebbb2830d5c73a5120a53ffbd5c.tar.gz bcm5719-llvm-1c9320cd4d970ebbb2830d5c73a5120a53ffbd5c.zip | |
Revert r310857 due to internal test failure
llvm-svn: 310907
Diffstat (limited to 'compiler-rt')
| -rw-r--r-- | compiler-rt/lib/profile/InstrProfilingFile.c | 28 | ||||
| -rw-r--r-- | compiler-rt/lib/profile/InstrProfilingUtil.c | 3 | ||||
| -rw-r--r-- | compiler-rt/test/profile/instrprof-path.c | 18 |
3 files changed, 13 insertions, 36 deletions
diff --git a/compiler-rt/lib/profile/InstrProfilingFile.c b/compiler-rt/lib/profile/InstrProfilingFile.c index ddfa3922569..d038bb9cb5b 100644 --- a/compiler-rt/lib/profile/InstrProfilingFile.c +++ b/compiler-rt/lib/profile/InstrProfilingFile.c @@ -513,29 +513,23 @@ const char *__llvm_profile_get_path_prefix(void) { COMPILER_RT_VISIBILITY void __llvm_profile_initialize_file(void) { const char *EnvFilenamePat; + const char *SelectedPat = NULL; + ProfileNameSpecifier PNS = PNS_unknown; int hasCommandLineOverrider = (INSTR_PROF_PROFILE_NAME_VAR[0] != 0); EnvFilenamePat = getFilenamePatFromEnv(); - if (EnvFilenamePat) - parseAndSetFilename(EnvFilenamePat, PNS_environment, 0); - else if (hasCommandLineOverrider) { - const char *SelectedPat = INSTR_PROF_PROFILE_NAME_VAR; - - size_t PrefixLen; - int StripLen; - const char *Prefix = lprofGetPathPrefix(&StripLen, &PrefixLen); - if (Prefix != NULL) { - char *StripPat = - COMPILER_RT_ALLOCA(PrefixLen + 1 + strlen(SelectedPat) + 1); - lprofApplyPathPrefix(StripPat, SelectedPat, Prefix, PrefixLen, StripLen); - SelectedPat = StripPat; - } - - parseAndSetFilename(SelectedPat, PNS_command_line, Prefix ? 1 : 0); + if (EnvFilenamePat) { + SelectedPat = EnvFilenamePat; + PNS = PNS_environment; + } else if (hasCommandLineOverrider) { + SelectedPat = INSTR_PROF_PROFILE_NAME_VAR; + PNS = PNS_command_line; } else { - parseAndSetFilename(NULL, PNS_default, 0); + SelectedPat = NULL; + PNS = PNS_default; } + parseAndSetFilename(SelectedPat, PNS, 0); } /* This API is directly called by the user application code. It has the diff --git a/compiler-rt/lib/profile/InstrProfilingUtil.c b/compiler-rt/lib/profile/InstrProfilingUtil.c index bed97e52689..fb68f30a5e1 100644 --- a/compiler-rt/lib/profile/InstrProfilingUtil.c +++ b/compiler-rt/lib/profile/InstrProfilingUtil.c @@ -196,8 +196,7 @@ lprofApplyPathPrefix(char *Dest, const char *PathStr, const char *Prefix, memcpy(Dest, Prefix, PrefixLen); - if (!IS_DIR_SEPARATOR(Prefix[PrefixLen - 1]) && - !IS_DIR_SEPARATOR(StrippedPathStr[0])) + if (!IS_DIR_SEPARATOR(Prefix[PrefixLen - 1])) Dest[PrefixLen++] = DIR_SEPARATOR; memcpy(Dest + PrefixLen, StrippedPathStr, strlen(StrippedPathStr) + 1); diff --git a/compiler-rt/test/profile/instrprof-path.c b/compiler-rt/test/profile/instrprof-path.c index ff0586b8dd2..28ee8ad0a48 100644 --- a/compiler-rt/test/profile/instrprof-path.c +++ b/compiler-rt/test/profile/instrprof-path.c @@ -1,8 +1,6 @@ // RUN: %clang_pgogen -O2 -o %t.0 %s // RUN: %clang_pgogen=%t.d1 -O2 -o %t.1 %s // RUN: %clang_pgogen=%t.d1/%t.d2 -O2 -o %t.2 %s -// RUN: %clang_pgogen=a/b/c/d -O2 -o %t.3 %s -// RUN: %clang_pgogen=/a/b/c/d -O2 -o %t.4 %s // // RUN: %run %t.0 "" // RUN: env LLVM_PROFILE_FILE=%t.d1/default.profraw %run %t.0 %t.d1/ @@ -11,17 +9,6 @@ // RUN: %run %t.2 %t.d1/%t.d2/ // RUN: %run %t.2 %t.d1/%t.d2/ %t.d1/%t.d2/%t.d3/blah.profraw %t.d1/%t.d2/%t.d3/ -// RUN: env GCOV_PREFIX=%t.prefix %run %t.3 %t.prefix/a/b/c/d/ -// RUN: env GCOV_PREFIX=%t.prefix env GCOV_PREFIX_STRIP=1 %run %t.3 %t.prefix/b/c/d/ -// RUN: env GCOV_PREFIX=%t.prefix env GCOV_PREFIX_STRIP=2 %run %t.3 %t.prefix/c/d/ -// RUN: env GCOV_PREFIX=%t.prefix env GCOV_PREFIX_STRIP=3 %run %t.3 %t.prefix/d/ - -// RUN: env GCOV_PREFIX=%t.prefix %run %t.4 %t.prefix/a/b/c/d/ -// RUN: env GCOV_PREFIX=%t.prefix env GCOV_PREFIX_STRIP=1 %run %t.4 %t.prefix/b/c/d/ -// RUN: env GCOV_PREFIX=%t.prefix env GCOV_PREFIX_STRIP=2 %run %t.4 %t.prefix/c/d/ -// RUN: env GCOV_PREFIX=%t.prefix env GCOV_PREFIX_STRIP=3 %run %t.4 %t.prefix/d/ - -#include <stdio.h> #include <string.h> const char *__llvm_profile_get_path_prefix(); @@ -37,11 +24,8 @@ int main(int argc, const char *argv[]) { expected = argv[1]; prefix = __llvm_profile_get_path_prefix(); - if (strcmp(prefix, expected)) { - fprintf(stderr, "Expected = %s\n", expected); - fprintf(stderr, " Actual = %s\n", prefix); + if (strcmp(prefix, expected)) return 1; - } if (argc == 4) { __llvm_profile_set_filename(argv[2]); |

