forked from OSchip/llvm-project
InstrProf: Use the same names for variables as we use in the profile
There's no need to use different names for the local variables than we use in the profile itself, and it's a bit simpler and easier to debug if we're consistent. llvm-svn: 223173
This commit is contained in:
Justin Bogner
parent
ec7ebebe55
commit
111c6533c2
|
|
@ -27,7 +27,7 @@ using namespace CodeGen;
|
|||
|
||||
void CodeGenPGO::setFuncName(StringRef Name,
|
||||
llvm::GlobalValue::LinkageTypes Linkage) {
|
||||
RawFuncName = Name;
|
||||
StringRef RawFuncName = Name;
|
||||
|
||||
// Function names may be prefixed with a binary '1' to indicate
|
||||
// that the backend should not modify the symbols due to any platform
|
||||
|
|
@ -35,20 +35,17 @@ void CodeGenPGO::setFuncName(StringRef Name,
|
|||
if (RawFuncName[0] == '\1')
|
||||
RawFuncName = RawFuncName.substr(1);
|
||||
|
||||
if (!llvm::GlobalValue::isLocalLinkage(Linkage)) {
|
||||
PrefixedFuncName = RawFuncName;
|
||||
return;
|
||||
FuncName = RawFuncName;
|
||||
if (llvm::GlobalValue::isLocalLinkage(Linkage)) {
|
||||
// For local symbols, prepend the main file name to distinguish them.
|
||||
// Do not include the full path in the file name since there's no guarantee
|
||||
// that it will stay the same, e.g., if the files are checked out from
|
||||
// version control in different locations.
|
||||
if (CGM.getCodeGenOpts().MainFileName.empty())
|
||||
FuncName = FuncName.insert(0, "<unknown>:");
|
||||
else
|
||||
FuncName = FuncName.insert(0, CGM.getCodeGenOpts().MainFileName + ":");
|
||||
}
|
||||
|
||||
// For local symbols, prepend the main file name to distinguish them.
|
||||
// Do not include the full path in the file name since there's no guarantee
|
||||
// that it will stay the same, e.g., if the files are checked out from
|
||||
// version control in different locations.
|
||||
PrefixedFuncName = CGM.getCodeGenOpts().MainFileName;
|
||||
if (PrefixedFuncName.empty())
|
||||
PrefixedFuncName.assign("<unknown>");
|
||||
PrefixedFuncName.append(":");
|
||||
PrefixedFuncName.append(RawFuncName);
|
||||
}
|
||||
|
||||
void CodeGenPGO::setFuncName(llvm::Function *Fn) {
|
||||
|
|
|
|||
|
|
@ -31,8 +31,7 @@ class RegionCounter;
|
|||
class CodeGenPGO {
|
||||
private:
|
||||
CodeGenModule &CGM;
|
||||
std::string PrefixedFuncName;
|
||||
StringRef RawFuncName;
|
||||
std::string FuncName;
|
||||
llvm::GlobalValue::LinkageTypes VarLinkage;
|
||||
|
||||
unsigned NumRegionCounters;
|
||||
|
|
@ -60,9 +59,9 @@ public:
|
|||
|
||||
/// Get the string used to identify this function in the profile data.
|
||||
/// For functions with local linkage, this includes the main file name.
|
||||
StringRef getFuncName() const { return StringRef(PrefixedFuncName); }
|
||||
StringRef getFuncName() const { return StringRef(FuncName); }
|
||||
std::string getFuncVarName(StringRef VarName) const {
|
||||
return ("__llvm_profile_" + VarName + "_" + RawFuncName).str();
|
||||
return ("__llvm_profile_" + VarName + "_" + FuncName).str();
|
||||
}
|
||||
|
||||
/// Return the counter value of the current region.
|
||||
|
|
|
|||
|
|
@ -4,8 +4,8 @@
|
|||
// RUN: %clang_cc1 -triple x86_64-apple-macosx10.9 -main-file-name c-captured.c %s -o - -emit-llvm -fprofile-instr-use=%t.profdata | FileCheck -check-prefix=PGOUSE -check-prefix=PGOALL %s
|
||||
|
||||
// PGOGEN: @[[DCC:__llvm_profile_counters_debug_captured]] = hidden global [3 x i64] zeroinitializer
|
||||
// PGOGEN: @[[CSC:__llvm_profile_counters___captured_stmt]] = internal global [2 x i64] zeroinitializer
|
||||
// PGOGEN: @[[C1C:__llvm_profile_counters___captured_stmt1]] = internal global [3 x i64] zeroinitializer
|
||||
// PGOGEN: @[[CSC:"__llvm_profile_counters_c-captured.c:__captured_stmt"]] = internal global [2 x i64] zeroinitializer
|
||||
// PGOGEN: @[[C1C:"__llvm_profile_counters_c-captured.c:__captured_stmt1"]] = internal global [3 x i64] zeroinitializer
|
||||
|
||||
// PGOALL-LABEL: define void @debug_captured()
|
||||
// PGOGEN: store {{.*}} @[[DCC]], i64 0, i64 0
|
||||
|
|
|
|||
|
|
@ -18,7 +18,7 @@
|
|||
// PGOGEN: @[[BLC:__llvm_profile_counters_boolop_loops]] = hidden global [9 x i64] zeroinitializer
|
||||
// PGOGEN: @[[COC:__llvm_profile_counters_conditional_operator]] = hidden global [3 x i64] zeroinitializer
|
||||
// PGOGEN: @[[MAC:__llvm_profile_counters_main]] = hidden global [1 x i64] zeroinitializer
|
||||
// PGOGEN: @[[STC:__llvm_profile_counters_static_func]] = internal global [2 x i64] zeroinitializer
|
||||
// PGOGEN: @[[STC:"__llvm_profile_counters_c-general.c:static_func"]] = internal global [2 x i64] zeroinitializer
|
||||
|
||||
// PGOGEN-LABEL: @simple_loops()
|
||||
// PGOUSE-LABEL: @simple_loops()
|
||||
|
|
|
|||
|
|
@ -24,12 +24,12 @@ int main(void) {
|
|||
return 0;
|
||||
}
|
||||
|
||||
// CHECK: @__llvm_profile_counters_foo_internal = internal global [3 x i64] zeroinitializer, section "__DATA,__llvm_prf_cnts", align 8
|
||||
// CHECK: @__llvm_profile_name_foo_internal = internal constant [24 x i8] c"c-linkage.c:foo_internal", section "__DATA,__llvm_prf_names", align 1
|
||||
// CHECK: @__llvm_profile_data_foo_internal = internal constant { i32, i32, i64, i8*, i64* } { i32 24, i32 3, i64 {{[0-9]+}}, i8* getelementptr inbounds ([24 x i8]* @__llvm_profile_name_foo_internal, i32 0, i32 0), i64* getelementptr inbounds ([3 x i64]* @__llvm_profile_counters_foo_internal, i32 0, i32 0) }, section "__DATA,__llvm_prf_data", align 8
|
||||
// CHECK: @"__llvm_profile_counters_c-linkage.c:foo_internal" = internal global [3 x i64] zeroinitializer, section "__DATA,__llvm_prf_cnts", align 8
|
||||
// CHECK: @"__llvm_profile_name_c-linkage.c:foo_internal" = internal constant [24 x i8] c"c-linkage.c:foo_internal", section "__DATA,__llvm_prf_names", align 1
|
||||
// CHECK: @"__llvm_profile_data_c-linkage.c:foo_internal" = internal constant { i32, i32, i64, i8*, i64* } { i32 24, i32 3, i64 {{[0-9]+}}, i8* getelementptr inbounds ([24 x i8]* @"__llvm_profile_name_c-linkage.c:foo_internal", i32 0, i32 0), i64* getelementptr inbounds ([3 x i64]* @"__llvm_profile_counters_c-linkage.c:foo_internal", i32 0, i32 0) }, section "__DATA,__llvm_prf_data", align 8
|
||||
static void foo_internal(void) { if (0){} if (0){} }
|
||||
|
||||
// CHECK: @llvm.used = appending global [5 x i8*] [i8* bitcast (i32 ()* @__llvm_profile_runtime_user to i8*), i8* bitcast ({ i32, i32, i64, i8*, i64* }* @__llvm_profile_data_foo to i8*), i8* bitcast ({ i32, i32, i64, i8*, i64* }* @__llvm_profile_data_foo_weak to i8*), i8* bitcast ({ i32, i32, i64, i8*, i64* }* @__llvm_profile_data_main to i8*), i8* bitcast ({ i32, i32, i64, i8*, i64* }* @__llvm_profile_data_foo_internal to i8*)], section "llvm.metadata"
|
||||
// CHECK: @llvm.used = appending global [5 x i8*] [i8* bitcast (i32 ()* @__llvm_profile_runtime_user to i8*), i8* bitcast ({ i32, i32, i64, i8*, i64* }* @__llvm_profile_data_foo to i8*), i8* bitcast ({ i32, i32, i64, i8*, i64* }* @__llvm_profile_data_foo_weak to i8*), i8* bitcast ({ i32, i32, i64, i8*, i64* }* @__llvm_profile_data_main to i8*), i8* bitcast ({ i32, i32, i64, i8*, i64* }* @"__llvm_profile_data_c-linkage.c:foo_internal" to i8*)], section "llvm.metadata"
|
||||
|
||||
// CHECK: define linkonce_odr i32 @__llvm_profile_runtime_user() {{.*}} {
|
||||
// CHECK: %[[REG:.*]] = load i32* @__llvm_profile_runtime
|
||||
|
|
|
|||
|
|
@ -11,7 +11,7 @@
|
|||
|
||||
// PGOGEN: @[[LWC:__llvm_profile_counters__Z7lambdasv]] = hidden global [4 x i64] zeroinitializer
|
||||
// PGOGEN: @[[MAC:__llvm_profile_counters_main]] = hidden global [1 x i64] zeroinitializer
|
||||
// LMBGEN: @[[LFC:"__llvm_profile_counters__ZZ7lambdasvENK3\$_0clEi"]] = internal global [3 x i64] zeroinitializer
|
||||
// LMBGEN: @[[LFC:"__llvm_profile_counters_cxx-lambda.cpp:_ZZ7lambdasvENK3\$_0clEi"]] = internal global [3 x i64] zeroinitializer
|
||||
|
||||
// PGOGEN-LABEL: define void @_Z7lambdasv()
|
||||
// PGOUSE-LABEL: define void @_Z7lambdasv()
|
||||
|
|
|
|||
|
|
@ -31,8 +31,8 @@ struct NSFastEnumerationState;
|
|||
@end;
|
||||
#endif
|
||||
|
||||
// PGOGEN: @[[FRC:"__llvm_profile_counters_\+\[A foreach:\]"]] = internal global [2 x i64] zeroinitializer
|
||||
// PGOGEN: @[[BLC:"__llvm_profile_counters___13\+\[A foreach:\]_block_invoke"]] = internal global [2 x i64] zeroinitializer
|
||||
// PGOGEN: @[[FRC:"__llvm_profile_counters_objc-general.m:\+\[A foreach:\]"]] = internal global [2 x i64] zeroinitializer
|
||||
// PGOGEN: @[[BLC:"__llvm_profile_counters_objc-general.m:__13\+\[A foreach:\]_block_invoke"]] = internal global [2 x i64] zeroinitializer
|
||||
// PGOGEN: @[[MAC:__llvm_profile_counters_main]] = hidden global [1 x i64] zeroinitializer
|
||||
|
||||
@interface A : NSObject
|
||||
|
|
|
|||
Loading…
Reference in New Issue