[PATCH] D79210: Let clang print registered targets for --version

Thu May 7 08:29:46 PDT 2020

yaxunl added a comment.

In D79210#2025058 <https://reviews.llvm.org/D79210#2025058>, @thakis wrote:

> In D79210#2025009 <https://reviews.llvm.org/D79210#2025009>, @thakis wrote:
>
> > We've talked about this 3 years ago in https://reviews.llvm.org/D33900 and back then decided that this should go behind a different flag. Has anything changed since then?
>
>
> To expand a bit:
>
> - Listing targets under `--version` doesn't make obvious sense semantically; a different `--print-target` flags would make more sense
> - Several tools parse --version / -v output, so adding it there might break them
>
>   From the description it sounds like a separate flag for this would work for you, so I'd say we revert this and reland it under a different flag. Does that make sense?

I am not aware of the previous discussion. I will revert it and reland it under `--print-target`.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79210