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

Max Moroz via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 9 12:55:46 PDT 2020


Dor1s added a comment.

> To be more constructive, do you have a use case where this option or its inverse is needed?

Yes, removal of this argument did break one pretty large system used by different projects / companies. We had to investigate and hotfix it on our end, but my point here that in general it's better not to break compatibility of the existing interfaces (doesn't matter whether we're talking about a cmd utility or an API), unless there's a very strong reason to do so.


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