[PATCH] D104363: [llvm] Mark more internal command line optins as cl::Hidden

[Timm Bäder via Phabricator via llvm-commits]

tbaeder added a comment.

The email thread about the llvm busybox implementation contains this paragraph:

> One implementation detail we think will be an issue is merging arguments in individual tools that use `cl::opt`. `cl::opt` works by maintaining a global state of flags, but we aren’t confident of what the resulting behavior will be when merging them together in the dispatching `main`. What we would like to avoid is having flags used by one specific tool available on other tools. To address this issue, we would like to migrate all tools to use `OptTable` which doesn't have this issue and has been the general direction most tools have been already moving into.

So, should all tools use `OptTable` instead?


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

https://reviews.llvm.org/D104363