mgorny created this revision. mgorny added reviewers: zturner, labath, k8stone. mgorny added a subscriber: lldb-commits.
Avoid including private LLVM config.h when not using MSVC. The file is used only to determine whether DIA SDK is available, and this can be true only when using MSVC. Therefore, short-circuit it to false on non-MSVC platforms to avoid the #include and fix standalone builds on those platforms. https://reviews.llvm.org/D26249 Files: unittests/SymbolFile/PDB/SymbolFilePDBTests.cpp Index: unittests/SymbolFile/PDB/SymbolFilePDBTests.cpp =================================================================== --- unittests/SymbolFile/PDB/SymbolFilePDBTests.cpp +++ unittests/SymbolFile/PDB/SymbolFilePDBTests.cpp @@ -10,7 +10,11 @@ #include "gtest/gtest.h" #include "llvm/ADT/STLExtras.h" -#include "llvm/Config/config.h" +#ifdef _MSC_VER // DIA SDK is available only when using MSVC +# include "llvm/Config/config.h" +#else +# define HAVE_DIA_SDK 0 +#endif #include "llvm/DebugInfo/PDB/PDBSymbolData.h" #include "llvm/DebugInfo/PDB/PDBSymbolExe.h" #include "llvm/Support/FileSystem.h"
Index: unittests/SymbolFile/PDB/SymbolFilePDBTests.cpp =================================================================== --- unittests/SymbolFile/PDB/SymbolFilePDBTests.cpp +++ unittests/SymbolFile/PDB/SymbolFilePDBTests.cpp @@ -10,7 +10,11 @@ #include "gtest/gtest.h" #include "llvm/ADT/STLExtras.h" -#include "llvm/Config/config.h" +#ifdef _MSC_VER // DIA SDK is available only when using MSVC +# include "llvm/Config/config.h" +#else +# define HAVE_DIA_SDK 0 +#endif #include "llvm/DebugInfo/PDB/PDBSymbolData.h" #include "llvm/DebugInfo/PDB/PDBSymbolExe.h" #include "llvm/Support/FileSystem.h"
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
