[PATCH] D39355: [dsymutil] Implement the --threads option
Jonas Devlieghere via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Nov 1 09:18:43 PDT 2017
JDevlieghere added a comment.
In https://reviews.llvm.org/D39355#912745, @aprantl wrote:
> How does the conflict manifest? The option registry shouldn't conflict because dsymutil's option is in a separate category. Or is it the alias that is conflicting? It would be good to understand this. Renaming the option would break compatibility for existing users of dsymutil, so we should avoid that.
It's definitely the `threads` option that conflict. Unfortunately you cannot have two options with the same name, even if they belong to a different category.
As discussed offline, I'll rename the options, add aliases for the internal version and mark them as deprecated/for compatibility.
Repository:
rL LLVM
https://reviews.llvm.org/D39355
More information about the llvm-commits
mailing list