<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Dec 8, 2015 at 10:51 AM, Peter Collingbourne <span dir="ltr"><<a href="mailto:peter@pcc.me.uk" target="_blank">peter@pcc.me.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Dec 08, 2015 at 06:39:56PM -0000, Rui Ueyama via llvm-commits wrote:<br>
> 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>
><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>
<br>
</span>Could you test this with a Windows-only test that runs the MSVC linker after<br>
running LLD on a small input file?</blockquote><div><br></div><div>I tried that, but looks like the environment in which lit tests run has "link" command of the GNU coreutils, so I couldn't call link.exe.</div></div><br></div></div>