[PATCH] D19234: [llvm-pdbdump] Print a better error message when DIA fails to load a PDB

Nico Weber via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 18 16:59:56 PDT 2016


thakis accepted this revision.
thakis added a comment.
This revision is now accepted and ready to land.

Thanks!


================
Comment at: tools/llvm-pdbdump/llvm-pdbdump.cpp:630
@@ +629,3 @@
+              "that either LLVM was not compiled with MSVC, or your MSVC "
+              "installation is corrupt.\n";
+    return;
----------------
Is that true? I thought I read somewhere that express doesn't include the dia SDK. Not sure though.


http://reviews.llvm.org/D19234





More information about the llvm-commits mailing list