maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[PR] README: remove note about experimental status of instrumentation 

Summary:
Vasily Leonenko,
Advanced Software Technology Lab, Huawei

(cherry picked from FBD30092666)
This commit is contained in:
Vasily Leonenko 2021-06-25 16:27:47 +08:00 committed by Maksim Panchenko
parent 361f3b5576
commit 285ac26d16
2 changed files with 2 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
bolt/README.md
View File

@ -140,7 +140,7 @@ a VM or on hardware that does not support it, then you can use only sample
events, such as cycles. In this case, the quality of the profile information
would not be as good, and performance gains with BOLT are expected to be lower.
#### With instrumentation (experimental)
#### With instrumentation
If perf record is not available to you, you may collect profile by first
instrumenting the binary with BOLT and then running it.
@ -153,8 +153,7 @@ profile should be ready for you in `/tmp/prof.fdata` and you can skip
**Step 2**.
Run BOLT with the `-help` option and check the category "BOLT instrumentation
options" for a quick reference on instrumentation knobs. Instrumentation is
experimental and currently does not work for PIEs/SOs.
options" for a quick reference on instrumentation knobs.
### Step 2: Convert Profile to BOLT Format

5
bolt/src/RuntimeLibs/InstrumentationRuntimeLibrary.cpp
View File

@ -65,11 +65,6 @@ void InstrumentationRuntimeLibrary::adjustCommandLineOptions(
"DT_FINI to write the profile\n";
exit(1);
}
if (!BC.HasFixedLoadAddress) {
outs() << "BOLT-ERROR: shared object or position-independent executables "
"are not allowed in instrumentation mode\n";
exit(1);
}
}
void InstrumentationRuntimeLibrary::emitBinary(BinaryContext &BC,