<div dir="ltr">I think it was not a good idea to remove __VERSION__, but only because it actually breaks software (and not just Python). I don't think it's terribly important to preserve theoretically-pure compatibility at this point, but it is rather useful to keep practical compatibility.<div><br></div><div><div>I note a couple other changes in this area which might also be problematic.</div><div>- "4.2.1 Compatible" was removed from the beginning of __VERSION__. </div><div>- clang -dumpversion reports clang's version (e.g. "9.0.0", rather than always saying "4.2.1")</div><div><br></div><div>My inclination is that the former is very unlikely to break something (why would anyone use that string for anything other than human readable output, for which the new value is better?).</div><div><br></div><div>The latter, though, seems more risky, and I'm not sure there's much value to that change. Possibly it should be put back.<br></div><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jul 19, 2019 at 7:53 AM JF Bastien <<a href="mailto:jfbastien@apple.com" target="_blank">jfbastien@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><br><div><br><blockquote type="cite"><div>On Jul 19, 2019, at 1:50 PM, JF Bastien via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>> wrote:</div><br class="gmail-m_-4286581131322355373gmail-m_-6546144872589228683gmail-m_729309539028946889Apple-interchange-newline"><div><div><br><div><blockquote type="cite"><div>On Jul 17, 2019, at 2:13 PM, James Y Knight via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>> wrote:</div><br class="gmail-m_-4286581131322355373gmail-m_-6546144872589228683gmail-m_729309539028946889Apple-interchange-newline"><div><div dir="ltr">The reason has always been that changing it would undoubtedly break some software which uses features of newer GCCs that aren't implemented in Clang. And there are indeed some of those features, even if they are weird edge cases.</div></div></blockquote><div><br></div><div>What you’re saying leads me to believe that <a href="https://reviews.llvm.org/rL365962" target="_blank">https://reviews.llvm.org/rL365962</a> shouldn’t have been committed. Unless we now decide it’s OK to break code which check GCC quirks?</div></div></div></div></blockquote><div><br></div><div>To be clear, it was reverted, but we don’t really seem to have a standing policy. It was more “Python broke” that caused a revert.</div><div><br></div><br><blockquote type="cite"><div><div><div><blockquote type="cite"><div><div dir="ltr"><div>Also, by this point, Clang is a widely-enough used compiler, that I'd expect most maintained software to be attempting to support it explicitly -- best via __has_builtin/__has_feature/__has_attribute/etc tests as applicable, falling back to the GCC version check if those macros don't exist.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jul 17, 2019 at 7:22 AM Simon Atanasyan via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
Recently I get a request to implement in Clang a MIPS-related feature<br>
which exists in GCC pre 4.4 and removed in later versions. There is a<br>
rationale behind such strange request -- third-party software checks a<br>
compiler's compatibility using __GNUC__,__GNUC_MINOR__ macros and<br>
selects code dedicated for obsoleted version of GCC.<br>
<br>
As to me I would use for __GNUC__,__GNUC_MINOR__,__GNUC_PATCHLEVEL__<br>
macros defined in the InitializePredefinedMacros function the same<br>
values as a minimal GCC version required for building LLVM/Clang [1].<br>
Now it's 5.1.0.<br>
<br>
Is there any rationale to continue declare compatibility with old GCC 4.2.1?<br>
<br>
[1] <a href="https://llvm.org/docs/GettingStarted.html#software" rel="noreferrer" target="_blank">https://llvm.org/docs/GettingStarted.html#software</a><br>
<br>
-- <br>
Simon Atanasyan<br>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
</blockquote></div>
_______________________________________________<br>cfe-dev mailing list<br><a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br><a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br></div></blockquote></div><br></div>_______________________________________________<br>cfe-dev mailing list<br><a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br><a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br></div></blockquote></div><br></div></blockquote></div>