[PATCH] D56342: [llvm-profdata] add value-cutoff functionality to show command

[David Li via Phabricator via llvm-commits]

davidxl added inline comments.


================
Comment at: docs/CommandGuide/llvm-profdata.rst:221
+
+ Show only those functions whose max count value are greater or equal to ``n``.
+ By default, the value-cutoff is set to 0.
----------------
value --> values


================
Comment at: docs/CommandGuide/llvm-profdata.rst:226
+
+ Only output names of functions whose max count value are below the cutoff
+ value.
----------------
value -->values


================
Comment at: tools/llvm-profdata/llvm-profdata.cpp:650
+      OS << ", " << format("%10" PRId64, VD[V].Count) << " ] ("
+         << format("%6.2f%%", (VD[V].Count * 100.0 / SiteSum)) << ")\n";
     }
----------------
just use %.2f%% to remove the extra space 


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56342/new/

https://reviews.llvm.org/D56342