<div dir="ltr"><div dir="ltr"><div dir="ltr">On Wed, Oct 30, 2019 at 11:53 PM Johan Engelen <<a href="mailto:jbc.engelen@gmail.com">jbc.engelen@gmail.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">On Thu, Oct 31, 2019 at 3:46 AM Saleem Abdulrasool <<a href="mailto:compnerd@compnerd.org" target="_blank">compnerd@compnerd.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">On Wed, Oct 30, 2019 at 4:17 PM Johan Engelen <<a href="mailto:jbc.engelen@gmail.com" target="_blank">jbc.engelen@gmail.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi all,<br><div>  Since  <span style="color:rgb(68,77,86);font-family:SFMono-Regular,Consolas,"Liberation Mono",Menlo,monospace;font-size:12px;text-align:right;white-space:nowrap">2724d9e12960cc1d93eeabbfc9aa1bffffa041cc</span>, llvm-config -cxxflags no longer reports the C++ language standard version (e.g. "-std=c++14") used to compile LLVM. This may break builds of projects that build with the same flags as LLVM, e.g. the LDC compiler.</div><div>Quoting from llvm/tools/llvm-config/CMakeLists.txt:</div><div>```</div><div># The language standard potentially affects the ABI/API of LLVM, so we want<br># to make sure it is reported by llvm-config.<br></div><div>```</div><div>Even in the case of default C++ version, in my opinion llvm-config should explicitly report it.</div></div></blockquote><div><br></div><div>I will add an option to report the C++ version, though that would be a new option and require reworking the consumers to be aware of that.  It would be better to actually rely on CMake to provide the details of the build rather than `llvm-config`.</div><div></div></div></div></blockquote><div> </div><div>Note that this is about the details for the build of a project that uses LLVM. I don't think projects should be forced to use CMake for setting up a build when they link to LLVM. The new option would help, but I feel it is better to be have the C++ version be reported by default and that a new option would disable it. In most cases I think one would want the c++ version to be specified because otherwise the project will be forced to compile with the same compiler and same compiler version as the LLVM package because things rely on compiler defaults to be the same. Hence the comment in llvm-config/CMakeLists.</div></div></div></blockquote><div><br></div><div>As of 7a25de6d2073f77ea1a0d8f4cd715bec55678941, `llvm-config` should have the standard flag in the reported C++ flags again.  Sorry about the late follow up, I intended to do that post-commit, and for the breakage.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div>regards,</div><div>  Johan</div></div></div></blockquote><div> </div></div>-- <br><div dir="ltr" class="gmail_signature">Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org</div></div></div>