[PATCH] D135127: [llvm-profdata] Add --output-format option

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 4 23:19:43 PDT 2022


davidxl added a comment.

1. the CommandLine documentation file needs to be updated.
2. what is the use case of splitting text and text-encoding?

3. Please also add test cases.



================
Comment at: llvm/tools/llvm-profdata/llvm-profdata.cpp:2743
+          clEnumValN(ShowOutputFormat::Text, "text", "print text output"),
+          clEnumValN(ShowOutputFormat::TextEncoding, "test-encoding",
+                     "emit text encoding"),
----------------
fix typo "test-encoding'


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135127



More information about the llvm-commits mailing list