[PATCH] D22593: [Profile] document %h and %m

Sean Silva via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 20 15:37:46 PDT 2016


silvas added a comment.

LGTM with some small wording nits.

We may want to extend this to mention number modifier to `%m` (e.g. `%4m`). Perhaps it is better to leave that to more advanced documentation -- your experiments showed that even just 1 merge pool is quite scalable IIRC.


================
Comment at: docs/UsersManual.rst:1500
@@ +1499,3 @@
+   name.  When this specifier is used, the profiler runtime will substitute ``%m``
+   with a unique integer identifier associated with the instrumented binary. Multiple
+   profiles dumped from different processes (running on the same or different hosts)
----------------
I would suggest saying `Additionally, multiple...` instead of just `Multiple` to highlight that this is an additional behavior that is caused by using `%m`.

================
Comment at: docs/UsersManual.rst:1501
@@ +1500,3 @@
+   with a unique integer identifier associated with the instrumented binary. Multiple
+   profiles dumped from different processes (running on the same or different hosts)
+   will be automatically merged by the profiler runtime during the dumping. If
----------------
You probably want to say "raw profiles" instead of just "profile" to clarify that this is a different sense of "merge" than `llvm-profdata merge ...` (or otherwise clarify).


https://reviews.llvm.org/D22593





More information about the cfe-commits mailing list