zturner accepted this revision. zturner added a comment. This revision is now accepted and ready to land.
looks good, sorry for the delay was busy with the LLVM dev conference. ================ Comment at: unittests/SymbolFile/DWARF/SymbolFileDWARFTests.cpp:33-36 +#if defined(_MSC_VER) +#include "lldb/Host/windows/windows.h" +#include <objbase.h> +#endif ---------------- This can go away, along with all the other preprocessor definitions. (only for the dwarf-specific test file). https://reviews.llvm.org/D26249 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits