[PATCH] D83530: WIP: [llvm-symbolizer] Switch command line parsing from llvm::cl to OptTable
Jordan Rupprecht via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 10 10:45:40 PDT 2020
rupprecht added a reviewer: rupprecht.
rupprecht added a comment.
`-f` is a GNU option, so we should keep that for compatibility. Is there a reason it's being removed?
... actually, the patch says `-f` is dropped, but from the tablegen, it looks like it's still there?
LGTM to the general idea of replacing cl::opt with tablegen options for simple command line tools. Will take a look when the grouped option issue is addressed.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83530/new/
https://reviews.llvm.org/D83530
More information about the llvm-commits
mailing list