[PATCH] D146899: [Docs] Added llvm-reduce docs in commandGuide

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 27 10:46:52 PDT 2023


dblaikie added inline comments.


================
Comment at: llvm/docs/CommandGuide/llvm-reduce.rst:9
+
+:program:`llvm-reduce` [*options*] [*input...*]
+
----------------



================
Comment at: llvm/docs/CommandGuide/llvm-reduce.rst:14
+
+The :program:`llvm-reduce` tool project that can be used for reducing the size of LLVM test cases.
+It works by removing redundant or unnecessary code from LLVM test cases while still preserving 
----------------
This sentence feels a bit awkward - the "The x tool project that can" maybe could be modeled after https://llvm.org/docs/CommandGuide/llvm-dwarfdump.html for instance, like:

"llvm-reduce reduces the size of LLVM test cases"

And might be worth mentioning tools like creduce, cvise, delta/multidelta as comparisons/similar tools/interfaces?


================
Comment at: llvm/docs/CommandGuide/llvm-reduce.rst:21
+
+LLVM-Reduce is a useful tool for reducing the size and 
+complexity of LLVM test cases, making it easier to identify and debug issues in 
----------------
/probably/ use ":program:`llvm-reduce`" as before/in the first paragraph rather than trying to figure out how to capitalize the tool name in a sentence


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146899



More information about the llvm-commits mailing list