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

[Jan Ole Hüser via Phabricator via llvm-commits]

j0le added a comment.

In D156436#4540180 <https://reviews.llvm.org/D156436#4540180>, @MaskRay wrote:

> Haven't looked closely(on my phone), but the new behavior makes sense.

I'm glad to hear that ☺.

In D156436#4540180 <https://reviews.llvm.org/D156436#4540180>, @MaskRay wrote:

> We probably need a unit test in llvm.

Seems like we already have a unit test for Flang:

  Failed Tests (2):
    Flang :: Driver/driver-help-hidden.f90
    Flang :: Driver/driver-help.f90

Sorry, that I didn't noticed earlier. I will look into it tomorrow.

I said “unit test for Flang”, but is it a unit test or regression test?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156436