[Lldb-commits] [PATCH] D18848: Add PDBASTParser and parse type information from PDB

[Jonas Hahnfeld via lldb-commits]

Hahnfeld added a comment.

In http://reviews.llvm.org/D18848#406521, @labath wrote:

> In http://reviews.llvm.org/D18848#406509, @zturner wrote:
>
> > I was under the impression the make build was officially unsupported/dead?
>
>
> You can still cmake to generate makefiles (and it does that by default).
>
> In any case, this problem should not be related to make, unit tests fail even when using ninja:
>
>   Error copying file "llvm/tools/lldb/unittests/SymbolFile/PDB/Inputs/test-pdb-types.exe" to "build/dbg/tools/lldb/unittests/SymbolFile/PDB/Inputs".
>


Yes, that's what I'm seeing here as well. Sorry for not posting the error message

In http://reviews.llvm.org/D18848#406539, @zturner wrote:

> Hmm, ok i will check it out later. I'm not sure why it's even trying to
>  copy an exe, should be only doing the pdb. Exe isn't even checked in



================
Comment at: unittests/SymbolFile/PDB/CMakeLists.txt:9
@@ +8,3 @@
+   test-dwarf.exe
+   test-pdb-types.exe
+   test-pdb-types.pdb)
----------------
Maybe because it is specified here and not compiled?


http://reviews.llvm.org/D18848