[PATCH] D33900: Print registered targets in clang's version information

Hans Wennborg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 27 09:23:13 PDT 2017


hans added a comment.

In https://reviews.llvm.org/D33900#822313, @mehdi_amini wrote:

> In https://reviews.llvm.org/D33900#820281, @hans wrote:
>
> > In https://reviews.llvm.org/D33900#818968, @mehdi_amini wrote:
> >
> > > I think @thakis is right: this too verbose to be the default --version.
> > >  We likely shouldn't ship this in clang-5.0 (@hans).
> >
> >
> > Let me know if you figure out a solution here and I'll merge it.
>
>
> I looped you in because the easy short term answer is: that we take our time to figure what to do in trunk but in the meantime we just revert from the release branch.


Makes sense. I've reverted r304899 and r304836 in r309285, and r304835 in r309286.


https://reviews.llvm.org/D33900





More information about the cfe-commits mailing list