This is an archive of the discontinued LLVM Phabricator instance.

Differential D13328

[PGO]: Eliminate __llvm_profile_register calls for Linux (LLVM changes)
ClosedPublic

Authored by davidxl on Sep 30 2015, 10:52 PM.

Details

Reviewers
davidxl
bogner
Summary

This is patch-2.

Diff Detail

Event Timeline

davidxl updated this revision to Diff 36188.Sep 30 2015, 10:52 PM
davidxl retitled this revision from to [PGO]: Eliminate __llvm_profile_register calls for Linux (LLVM changes).
davidxl updated this object.
davidxl added a reviewer: bogner.
davidxl added a subscriber: llvm-commits.
davidxl accepted this revision.Oct 13 2015, 12:11 PM
davidxl added a reviewer: davidxl.
This revision is now accepted and ready to land.Oct 13 2015, 12:11 PM
davidxl closed this revision.Oct 13 2015, 12:11 PM

Revision Contents

PathSize
lib/
Transforms/
Instrumentation/
3 lines
test/
Instrumentation/
InstrProfiling/
4 lines

Diff 36188

lib/Transforms/Instrumentation/InstrProfiling.cpp

Show First 20 Lines • Show All 251 Lines • ▼ Show 20 LinesInstrProfiling::getOrCreateRegionCounters(InstrProfIncrementInst *Inc) {
return Counters; return Counters;
} }
void InstrProfiling::emitRegistration() { void InstrProfiling::emitRegistration() {
// Don't do this for Darwin. compiler-rt uses linker magic. // Don't do this for Darwin. compiler-rt uses linker magic.
if (Triple(M->getTargetTriple()).isOSDarwin()) if (Triple(M->getTargetTriple()).isOSDarwin())
return; return;
// Use linker script magic to get data/cnts/name start/end.
if (Triple(M->getTargetTriple()).isOSLinux()) return;
// Construct the function. // Construct the function.
auto *VoidTy = Type::getVoidTy(M->getContext()); auto *VoidTy = Type::getVoidTy(M->getContext());
auto *VoidPtrTy = Type::getInt8PtrTy(M->getContext()); auto *VoidPtrTy = Type::getInt8PtrTy(M->getContext());
auto *RegisterFTy = FunctionType::get(VoidTy, false); auto *RegisterFTy = FunctionType::get(VoidTy, false);
auto *RegisterF = Function::Create(RegisterFTy, GlobalValue::InternalLinkage, auto *RegisterF = Function::Create(RegisterFTy, GlobalValue::InternalLinkage,
"__llvm_profile_register_functions", M); "__llvm_profile_register_functions", M);
RegisterF->setUnnamedAddr(true); RegisterF->setUnnamedAddr(true);
if (Options.NoRedZone) if (Options.NoRedZone)
▲ Show 20 Lines • Show All 114 Lines • Show Last 20 Lines

test/Instrumentation/InstrProfiling/platform.ll

Show All 17 Lines
} }
declare void @llvm.instrprof.increment(i8*, i64, i32, i32) declare void @llvm.instrprof.increment(i8*, i64, i32, i32)
;; Emit registration functions for platforms that don't find the ;; Emit registration functions for platforms that don't find the
;; symbols by their sections. ;; symbols by their sections.
; MACHO-NOT: define internal void @__llvm_profile_register_functions ; MACHO-NOT: define internal void @__llvm_profile_register_functions
; ELF: define internal void @__llvm_profile_register_functions ; ELF-NOT: define internal void @__llvm_profile_register_functions
; MACHO-NOT: define internal void @__llvm_profile_init ; MACHO-NOT: define internal void @__llvm_profile_init
; ELF: define internal void @__llvm_profile_init ; ELF-NOT: define internal void @__llvm_profile_init