rnk added a comment.

In D56803#1441876 <https://reviews.llvm.org/D56803#1441876>, @sylvestre.ledru 
wrote:

> @rnk btw, do you think it should be added to the clang release notes?


I think it is release-note worthy. If you change __VERSION__, you can mention 
that as well.



================
Comment at: lib/Driver/Driver.cpp:1631
-    // introduce a non-pedantically GCC compatible mode to Clang in which we
-    // provide sensible definitions for -dumpversion, __VERSION__, etc.
-    llvm::outs() << "4.2.1\n";
----------------
sylvestre.ledru wrote:
> rnk wrote:
> > We really ought to reconsider `__VERSION__` as well, although I don't think 
> > anyone checks that. Today I get this for it:
> >   "4.2.1 Compatible Clang 9.0.0 (g...@github.com:llvm/llvm-project.git 
> > bd6354105b055af6193fb2b7561fe6f5ae0eb0d8)"
> I can have a look to that if you want
If you're feeling motivated, I think it's worth doing to try to get us into a 
more consistent state.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56803



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

Reply via email to