From 0971eb7364bde89cacbfceebad94a66a638c5046 Mon Sep 17 00:00:00 2001 From: Xinliang David Li Date: Mon, 19 Oct 2015 04:17:10 +0000 Subject: [PATCH] [PGO] Eliminate prof data register calls on FreeBSD platform This is a follow up patch of r250199 after verifying the start/stop section symbols work as spected on FreeBSD. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@250679 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/Instrumentation/InstrProfiling.cpp | 4 +++- test/Instrumentation/InstrProfiling/platform.ll | 12 ++++++++---- 2 files changed, 11 insertions(+), 5 deletions(-) diff --git a/lib/Transforms/Instrumentation/InstrProfiling.cpp b/lib/Transforms/Instrumentation/InstrProfiling.cpp index a92e7f8297b..a68fd3db6c2 100644 --- a/lib/Transforms/Instrumentation/InstrProfiling.cpp +++ b/lib/Transforms/Instrumentation/InstrProfiling.cpp @@ -258,7 +258,9 @@ void InstrProfiling::emitRegistration() { return; // Use linker script magic to get data/cnts/name start/end. - if (Triple(M->getTargetTriple()).isOSLinux()) return; + if (Triple(M->getTargetTriple()).isOSLinux() || + Triple(M->getTargetTriple()).isOSFreeBSD()) + return; // Construct the function. auto *VoidTy = Type::getVoidTy(M->getContext()); diff --git a/test/Instrumentation/InstrProfiling/platform.ll b/test/Instrumentation/InstrProfiling/platform.ll index e2c3839d4f1..9fe84b043e2 100644 --- a/test/Instrumentation/InstrProfiling/platform.ll +++ b/test/Instrumentation/InstrProfiling/platform.ll @@ -2,7 +2,8 @@ ; RUN: opt < %s -mtriple=x86_64-apple-macosx10.10.0 -instrprof -S | FileCheck %s -check-prefix=MACHO ; RUN: opt < %s -mtriple=x86_64-unknown-linux -instrprof -S | FileCheck %s -check-prefix=LINUX -; RUN: opt < %s -mtriple=x86_64-unknown-freebsd -instrprof -S | FileCheck %s -check-prefix=BSD +; RUN: opt < %s -mtriple=x86_64-unknown-freebsd -instrprof -S | FileCheck %s -check-prefix=FREEBSD +; RUN: opt < %s -mtriple=x86_64-pc-solaris -instrprof -S | FileCheck %s -check-prefix=SOLARIS @__llvm_profile_name_foo = hidden constant [3 x i8] c"foo" ; MACHO: @__llvm_profile_name_foo = hidden constant [3 x i8] c"foo", section "__DATA,__llvm_prf_names", align 1 @@ -13,7 +14,8 @@ ; MACHO: @__llvm_profile_data_foo = hidden constant {{.*}}, section "__DATA,__llvm_prf_data", align 8 ; LINUX: @__llvm_profile_data_foo = hidden constant {{.*}}, section "__llvm_prf_data", align 8 -; BSD: @__llvm_profile_data_foo = hidden constant {{.*}}, section "__llvm_prf_data", align 8 +; FREEBSD: @__llvm_profile_data_foo = hidden constant {{.*}}, section "__llvm_prf_data", align 8 +; SOLARIS: @__llvm_profile_data_foo = hidden constant {{.*}}, section "__llvm_prf_data", align 8 define void @foo() { call void @llvm.instrprof.increment(i8* getelementptr inbounds ([3 x i8], [3 x i8]* @__llvm_profile_name_foo, i32 0, i32 0), i64 0, i32 1, i32 0) ret void @@ -26,8 +28,10 @@ declare void @llvm.instrprof.increment(i8*, i64, i32, i32) ; MACHO-NOT: define internal void @__llvm_profile_register_functions ; LINUX-NOT: define internal void @__llvm_profile_register_functions -; BSD: define internal void @__llvm_profile_register_functions +; FREEBSD-NOT: define internal void @__llvm_profile_register_functions +; SOLARIS: define internal void @__llvm_profile_register_functions ; MACHO-NOT: define internal void @__llvm_profile_init ; LINUX-NOT: define internal void @__llvm_profile_init -; BSD: define internal void @__llvm_profile_init +; FREEBSD-NOT: define internal void @__llvm_profile_init +; SOLARIS: define internal void @__llvm_profile_init -- 2.11.0