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

Reply via email to