<div dir="ltr">Hi Chris and Saleem,<div>  I'm not sure about the exact proposal, but my initial thought is that using pkg-config instead of llvm-config would increase the burden for us because we would have to add a dependency on pkg-config on all platforms we support, including Windows where it is uncommon. We support building with LLVM 3.9 up to trunk, so there'd be additional hassle of figuring out which LLVM version a user has.</div><div>How we use llvm-config:</div><div>- allow user to explicitly which directory to look for LLVM (i.e. where to look for llvm-config belonging to that LLVM install)</div><div>- --libs + list of components to get link libraries list</div><div>- --host-target, --assertion-mode, --target-built, and several other LLVM-specific attributes</div><div><div><br></div><div>(we could probably also use LLVMConfig.cmake but instead we use our own 'FindLLVM' cmake script that finds and invokes llvm-config and then patches up compiler flags, etc, to make things work with LLVM 3.9-trunk and gcc/clang/msvc. I'm afraid LLVMConfig.cmake may do more than we want it to do; we might have to give up control over some things that we need control over.)</div><div><br></div><div>Thank you Saleem for including `-std=...` again in cxxflags.</div><div><br></div><div>-Johan</div><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Nov 2, 2019 at 5:16 PM Chris Bieneman <<a href="mailto:chris.bieneman@me.com">chris.bieneman@me.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 style="overflow-wrap: break-word;"><br><div>Johan,</div><div><br></div><div>Since llvm-config is something that matters to you I'm curious to get your opinion (and that of anyone else who can chime in) on something. At the dev meeting during the CMake roundtable we discussed deprecating llvm-config in favor of pkg-config.</div><div><br></div><div>Would LLVM supporting pkg-config meet your needs in place of llvm-config?</div><div><br></div><div>-Chris</div><div><br><blockquote type="cite"><div>On Oct 31, 2019, at 1:53 AM, Johan Engelen via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:</div><br><div><div dir="ltr" style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><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:1px solid 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:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi all,<br><div> <span> </span>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><br></div><div>regards,</div><div>  Johan</div><div> </div><div><br></div></div></div><span style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none;float:none;display:inline">_______________________________________________</span><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><span style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none;float:none;display:inline">LLVM Developers mailing list</span><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><a href="mailto:llvm-dev@lists.llvm.org" style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px" target="_blank">llvm-dev@lists.llvm.org</a><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none"><a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a></div></blockquote></div><br></div></blockquote></div>