<div dir="ltr">Actually LLDB does use DIA.  SymbolFilePDB enables simple symbolization of programs built with PDB debug info support on Windows and uses DIA to accomplish this.</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Nov 7, 2016 at 1:48 PM Michał Górny <<a href="mailto:mgorny@gentoo.org">mgorny@gentoo.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">mgorny added a comment.<br class="gmail_msg">
<br class="gmail_msg">
In <a href="https://reviews.llvm.org/D26255#588591" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D26255#588591</a>, @beanz wrote:<br class="gmail_msg">
<br class="gmail_msg">
> I think you're conflating two different types of variables, and I don't understand why.<br class="gmail_msg">
><br class="gmail_msg">
> `LLVM_ENABLE_*` variables are intended to be used for people to opt in or out of functionality. `HAVE_*` variables are for compiler checks against the system capabilities. Generally system capabilities shouldn't be captured in LLVM installed bits, which is what you're doing here.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
I was considering this issue, and I've decided having user control over DIA SDK a worthwhile future effort. While this patch doesn't add it, it prepares the stuff to use it when it's added.<br class="gmail_msg">
<br class="gmail_msg">
> I think the correct solution would be for you to perform the `HAVE_DIA_SDK` check in LLDB as well as LLVM. If the code is sufficiently complex, the LLVM version of the code could be abstracted into a module and re-used.<br class="gmail_msg">
<br class="gmail_msg">
DIA support enables some PDB debug format support in LLVM, that is unavailable otherwise. LLDB doesn't use DIA at all but the PDB tests require DIA support enabled during LLVM build. In this regard, it's similar to zlib support, upon which I based my patch.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D26255" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D26255</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>