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

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

LLVMOption used to be part of Clang and certain features are heavily coupled with Clang Driver behaviors.
Ideally we should have llvm/ tests (llvm/unittests/Option, or somewhere in llvm/test) for better layering (https://maskray.me/blog/2021-08-08-toolchain-testing#the-test-checks-at-the-wrong-layer), but if not applicable, somewhere in clang/unittests/Driver is fine as well.
Many contributors don't build flang and consider clang more "core", so having clang tests seems important if we don't have a good place in llvm/


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156436