[PATCH] D156436: [llvm/OptTable] Print options without documentation

Jan Ole Hüser via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 28 07:02:36 PDT 2023


j0le updated this revision to Diff 545142.
j0le added a comment.
Herald added a reviewer: jdoerfert.
Herald added a reviewer: sscalpone.
Herald added subscribers: jplehr, sstefan1.
Herald added a reviewer: awarzynski.

I added the missing undocumented options of Flang to the tests:

- flang/test/Driver/driver-help-hidden.f90
- flang/test/Driver/driver-help.f90

For the most part that are options starting with "-fno-".

I also changed quite a lot of white space in these tests, so that the help texts are aligned. I don't know if that's good or bad.
With `git show --ignore-all-space` one can see the relevant lines.

Furthermore I remove the comma, as suggested by @serge-sans-paille


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

https://reviews.llvm.org/D156436

Files:
  flang/test/Driver/driver-help-hidden.f90
  flang/test/Driver/driver-help.f90
  llvm/include/llvm/Option/OptTable.h
  llvm/lib/Option/OptTable.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D156436.545142.patch
Type: text/x-patch
Size: 42684 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230728/70a45e2e/attachment.bin>


More information about the llvm-commits mailing list