diff options
author | Xinliang David Li <davidxl@google.com> | 2017-08-14 16:51:05 +0000 |
---|---|---|
committer | Xinliang David Li <davidxl@google.com> | 2017-08-14 16:51:05 +0000 |
commit | 3caa058f838394a73e896a500784a21915f209ef (patch) | |
tree | 84e16f5b419acf7c1b93a314b562c3d7ec732561 | |
parent | 83db677110a0953642661a2e526521e30c358c53 (diff) |
[PGO] Add support for relocate profile dumping directory
Differential Revsion: http://reviews.llvm.org/D36648
git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@310857 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/profile/InstrProfilingFile.c | 28 | ||||
-rw-r--r-- | lib/profile/InstrProfilingUtil.c | 3 | ||||
-rw-r--r-- | test/profile/instrprof-path.c | 18 |
3 files changed, 36 insertions, 13 deletions
diff --git a/lib/profile/InstrProfilingFile.c b/lib/profile/InstrProfilingFile.c index d038bb9cb..ddfa39225 100644 --- a/lib/profile/InstrProfilingFile.c +++ b/lib/profile/InstrProfilingFile.c @@ -513,23 +513,29 @@ 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) { - SelectedPat = EnvFilenamePat; - PNS = PNS_environment; - } else if (hasCommandLineOverrider) { - SelectedPat = INSTR_PROF_PROFILE_NAME_VAR; - PNS = PNS_command_line; + 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); } else { - SelectedPat = NULL; - PNS = PNS_default; + parseAndSetFilename(NULL, PNS_default, 0); } - parseAndSetFilename(SelectedPat, PNS, 0); } /* This API is directly called by the user application code. It has the diff --git a/lib/profile/InstrProfilingUtil.c b/lib/profile/InstrProfilingUtil.c index fb68f30a5..bed97e526 100644 --- a/lib/profile/InstrProfilingUtil.c +++ b/lib/profile/InstrProfilingUtil.c @@ -196,7 +196,8 @@ lprofApplyPathPrefix(char *Dest, const char *PathStr, const char *Prefix, memcpy(Dest, Prefix, PrefixLen); - if (!IS_DIR_SEPARATOR(Prefix[PrefixLen - 1])) + if (!IS_DIR_SEPARATOR(Prefix[PrefixLen - 1]) && + !IS_DIR_SEPARATOR(StrippedPathStr[0])) Dest[PrefixLen++] = DIR_SEPARATOR; memcpy(Dest + PrefixLen, StrippedPathStr, strlen(StrippedPathStr) + 1); diff --git a/test/profile/instrprof-path.c b/test/profile/instrprof-path.c index 28ee8ad0a..ff0586b8d 100644 --- a/test/profile/instrprof-path.c +++ b/test/profile/instrprof-path.c @@ -1,6 +1,8 @@ // 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/ @@ -9,6 +11,17 @@ // 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(); @@ -24,8 +37,11 @@ int main(int argc, const char *argv[]) { expected = argv[1]; prefix = __llvm_profile_get_path_prefix(); - if (strcmp(prefix, expected)) + if (strcmp(prefix, expected)) { + fprintf(stderr, "Expected = %s\n", expected); + fprintf(stderr, " Actual = %s\n", prefix); return 1; + } if (argc == 4) { __llvm_profile_set_filename(argv[2]); |