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

[James Henderson via Phabricator via llvm-commits]

jhenderson accepted this revision.
jhenderson added a comment.
This revision is now accepted and ready to land.

Looks good, but this probably needs a release note to highlight the change in how to disable certain options, in case anybody was relying on the old behaviour.



================
Comment at: llvm/tools/llvm-symbolizer/llvm-symbolizer.cpp:203
+
+static opt::InputArgList parseOptions(int argc, char *argv[], bool IsAddr2Line,
+                                      StringSaver &Saver,
----------------
`Argc`, `Argv` here too.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83530