[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