Skip to content

Commit 81b4c1d

Browse files
committed
address review comments regarding PGO
1 parent 6edbe07 commit 81b4c1d

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

PCbuild/readme.txt

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,11 +79,18 @@ platform toolset, but LLVMToolsVersion has to be set accordingly.
7979
Setting the major version is enough, although you can be specific
8080
and use 18.1.8 in the above example, too.
8181

82-
Even --pgo works out of the box. However, if you want to run the PGO task
82+
Use the --pgo option to build with PGO (Profile Guided Optimization).
83+
84+
However, if you want to run the PGO task
8385
on a different host than the build host, you must pass
8486
"/p:CLANG_PROFILE_PATH=<relative-path-to-instrumented-dir-on-remote-host>"
8587
in the PGInstrument step to make sure the profile data is generated
8688
into the instrumented directory when running the PGO task.
89+
E.g., if you place the instrumented binaries into the folder
90+
"workdir/instrumented" and then run the PGO task using "workdir"
91+
as the current working directory, the usage is
92+
"/p:CLANG_PROFILE_PATH=workdir/instrumented"
93+
8794
Like in the MSVC case, after fetching (or manually copying) the instrumented
8895
folder back into your build tree, you can continue with the PGUpdate
8996
step with no further parameters.

0 commit comments

Comments
 (0)