<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Dec 8, 2015 at 11:00 AM David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Dec 8, 2015 at 10:39 AM, Rui Ueyama via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: ruiu<br>
Date: Tue Dec  8 12:39:55 2015<br>
New Revision: 255039<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=255039&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=255039&view=rev</a><br>
Log:<br>
COFF: Create an empty but valid PDF file.<br></blockquote><div><br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>^ had me wondering for a second there (PDF files? from the compiler?)</div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
MSVC linker considers PDB files created with this patch valid.<br>
So you don't have to remove PDB files created by lld before<br>
running MSVC linker.<br>
<br>
This patch has no test since llvm-pdbdump dislikes PDB files<br>
with no metadata streams.<br></blockquote><div><br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Could we fix the dumper so we can test this please?</div></div></div></div></blockquote><div>Dumper is using DIA SDK, and DIA SDK doesn't like PDB files with no metadata streams.  So this will only be fixed once we have a non-DIA based dumper, which is blocked on getting Dave B's codeview stuff in. </div><div><br></div><div>Once Rui or David M or myself (if I ever finish this Python 3 work) figures out how to write the PDB metadata stream, then I imagine pdb dumper will work.</div></div></div>