[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