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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
