[PATCH] D83530: [llvm-symbolizer] Switch command line parsing from llvm::cl to OptTable

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 12 00:27:25 PDT 2020


MaskRay added a comment.

In D83530#2144663 <https://reviews.llvm.org/D83530#2144663>, @rupprecht wrote:

> `-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?


Fixed. A subsequent patch can drop `-f=` and `-e=` which are not conventional. This patch tries hard to stick with the current behavior to prevent surprise.


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