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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to