summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorXinliang David Li <davidxl@google.com>2017-08-14 16:51:05 +0000
committerXinliang David Li <davidxl@google.com>2017-08-14 16:51:05 +0000
commit3caa058f838394a73e896a500784a21915f209ef (patch)
tree84e16f5b419acf7c1b93a314b562c3d7ec732561
parent83db677110a0953642661a2e526521e30c358c53 (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.c28
-rw-r--r--lib/profile/InstrProfilingUtil.c3
-rw-r--r--test/profile/instrprof-path.c18
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]);