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

Nico Weber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 7 07:54:17 PDT 2020


thakis added a comment.

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?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79210





More information about the cfe-commits mailing list