<div dir="ltr">LGTM<div><br></div><div>I'll commit it for you.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 11, 2017 at 1:48 PM, Shoaib Meenai <span dir="ltr"><<a href="mailto:smeenai@fb.com" target="_blank">smeenai@fb.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">







<div bgcolor="white" lang="EN-US" link="#0563C1" vlink="#954F72">
<div class="m_7789425998460600020WordSection1">
<p class="MsoNormal">CCing Rui.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">In my experience, patches usually get more visibility if you upload them to Phabricator.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<div style="border:none;border-top:solid #b5c4df 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:12.0pt;color:black">From: </span></b><span style="font-size:12.0pt;color:black">llvm-commits <<a href="mailto:llvm-commits-bounces@lists.llvm.org" target="_blank">llvm-commits-bounces@lists.<wbr>llvm.org</a>> on behalf of Michael Rickert via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>><br>
<b>Reply-To: </b>Michael Rickert <<a href="mailto:mike@maulingmonkey.com" target="_blank">mike@maulingmonkey.com</a>><br>
<b>Date: </b>Sunday, September 10, 2017 at 7:49 PM<br>
<b>To: </b>"<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>" <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>><br>
<b>Subject: </b>[lld] Small patch to ignore "/natvis:..." flag in the COFF / lld-link.exe frontend.<u></u><u></u></span></p>
</div><span class="">
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<p class="MsoNormal">MSVC’s link.exe supports an undocumented flag, /natvis:path/to/a/file.natvis, which will embed natvis files into generated pdbs – useful for dealing with multiple versions and ease of use when debugging (no IDE setup required!)<u></u><u></u></p>
<p class="MsoNormal"> <u></u><u></u></p>
<p class="MsoNormal">The attached patch simply makes LLVM’s lld-link.exe ignore this flag for now, fixing a recent rustc regression of mine: 
<a href="https://github.com/rust-lang/rust/pull/43221#issuecomment-328276585" target="_blank">https://github.com/rust-lang/<wbr>rust/pull/43221#issuecomment-<wbr>328276585</a> .  I intend to add a “temporary” workaround on rustc’s end as well.<u></u><u></u></p>
</span></div>
</div>

</blockquote></div><br></div>