<div dir="ltr">If you mean just the interface, then yes I can an effort to make sure the interface doesn't expose anything Windows-specific.  This particular implementation will obviously need to use Windows specific things though.<br><br><div class="gmail_quote">On Thu Jan 29 2015 at 11:00:38 AM Dennis Luehring <<a href="mailto:dl.soluz@gmx.net">dl.soluz@gmx.net</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">>The pdb reading code relies *very heavily* on Windows APIs<br>
<br>
can you make shure not to have any windows api stuff in the pdb-reader interface<br>
so its easy to - if someone wants to - implement an windows-api independent version (support reading under linux etc.)<br>
because the pdb format needs to be analysed anyway for writing pdb information on compile/link (in the future)<br>
<br>
<br>
Am 29.01.2015 um 19:20 schrieb Zachary Turner:<br>
> I've been working on adding pdb reading support to llvm.  This started as a<br>
> tool for dumping info from a pdb (similar to llvm-dwarfdump), which has<br>
> been checked in and currently has limited support for dumping pdb.<br>
><br>
> There's still more to be done on the pdb dumping tool, but at this point --<br>
> to reduce duplicated effort -- I think it makes the most sense to start<br>
> moving some of this logic into a library in llvm, and then change<br>
> llvm-pdbdump to use the library.  Later, once the library is more<br>
> comprehensive, I plan to then use it in LLDB for reading PDBs while<br>
> debugging on Windows.<br>
><br>
> I think the best way to do this is to move all of the code in lib/DebugInfo<br>
> to lib/DebugInfo/dwarf, and then make another folder called<br>
> lib/DebugInfo/pdb.   These would then be compiled into two separate<br>
> libraries.<br>
><br>
> Another approach is to just put the PDB code in the same folder as the<br>
> dwarf code, but I don't like this approach for a number of reasons:<br>
><br>
> 1) Not every consumer of DebugInfo wants both types of DebugInfo.<br>
> 2) The pdb reading code relies *very heavily* on Windows APIs, and will not<br>
> compile on other platforms.  This is solvable with some CMake machinery,<br>
> but it's ugly and unwarranted in my opinion.<br>
><br>
> So as a first step in this direction I'd like to propose moving the code in<br>
> lib/DebugInfo to lib/DebugInfo/dwarf, and then updating the rest of llvm<br>
> accordingly.<br>
><br>
> Thoughts?  Comments?  Suggestions?<br>
> Zach<br>
><br>
><br>
><br>
> ______________________________<u></u>_________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:LLVMdev@cs.uiuc.edu" target="_blank">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvmdev</a><br>
<br>
______________________________<u></u>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu" target="_blank">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvmdev</a><br>
</blockquote></div></div>