[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