-
Notifications
You must be signed in to change notification settings - Fork 14.3k
[Clang][PGO][UserManual] Specify usage of -b flag #139763
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
llvm-profgen cannot accept the perf profiles collected without `-b` and errors out with a message "Invalid perf script input!". This can also be found from code in function checkPerfScriptType() in tools/llvm-profgen/PerfReader.cpp.
@llvm/pr-subscribers-clang Author: Madhur Amilkanthwar (madhur13490) Changesllvm-profgen cannot accept the perf profiles collected without This can also be validated from the code in function Full diff: https://github.com/llvm/llvm-project/pull/139763.diff 1 Files Affected:
diff --git a/clang/docs/UsersManual.rst b/clang/docs/UsersManual.rst
index 6f804a10748d8..00899dcb8d8c2 100644
--- a/clang/docs/UsersManual.rst
+++ b/clang/docs/UsersManual.rst
@@ -2792,6 +2792,9 @@ usual build cycle when using sample profilers for optimization:
$ llvm-profgen --binary=./code --output=code.prof --perfdata=perf.data
+ Please note, ``perf.data`` must be collected with ``-b`` flag to Linux ``perf``
+ for the above step to work.
+
When using SEP the output is in the textual format corresponding to
``llvm-profgen --perfscript``. For example:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Looks good to me.
Step 2 mentions -b
, but makes it sound optional because it was optional with AutoFDO. Llvm-profgen doesn't have a mode to not require LBRs and this is good to point out.
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/174/builds/17728 Here is the relevant piece of the build log for the reference
|
llvm-profgen cannot accept the perf profiles collected without
-b
and errors out with a message"Invalid perf script input!"
.This can also be validated from the code in function
checkPerfScriptType()
intools/llvm-profgen/PerfReader.cpp.