[PATCH] D64504: Various minor tweaks to CLCompatOptions.td

Nico Weber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 10 09:04:38 PDT 2019


thakis created this revision.
thakis added a reviewer: rnk.

- Add back indentation I accidentally removed in r364901
- Wrap two lines to 80 cols
- Slightly tighten up help text for several flags
- Consistently use "Do not" instead of "Don't"
- Make every option description start with a verb
- Use "Set" instead of "Specify"
- Mark default values of options more consistently
- Remove text about "/Zi" not producing PDBs since it's confusing for people not intimately familiar with the implementation of the normal PDB pipeline. /Zi lets the linker produce PDBs, which is what most users want.
- Consistently use "file" over "filename" in meta var names, consistently use "file name" over "filename" in text
- Make all output setting options have consistent language

Hopefully makes help output a bit easier to read.
Also makes help for most flags in the "CL COMPAT FLAGS" section fit in an 80 column terminal.


https://reviews.llvm.org/D64504

Files:
  clang/include/clang/Driver/CLCompatOptions.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D64504.208977.patch
Type: text/x-patch
Size: 12581 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190710/0b5b49c1/attachment.bin>


More information about the cfe-commits mailing list