[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