<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 4, 2018 at 2:37 AM, 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" link="blue" vlink="purple" lang="EN-US">
<div class="m_3202043661778054723WordSection1">
<p class="MsoNormal">+Rui and Hans, in case they miss it on the list.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">(Side note: it's much easier to keep track of patches on Phabricator IMO.)</p></div></div></blockquote><div><br></div><div>Thank you Shoaib, I will try that next time. <br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div bgcolor="white" link="blue" vlink="purple" lang="EN-US"><div class="m_3202043661778054723WordSection1"><p class="MsoNormal"><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 Andrew Kelley via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>><br>
<b>Reply-To: </b>Andrew Kelley <<a href="mailto:superjoe30@gmail.com" target="_blank">superjoe30@gmail.com</a>><br>
<b>Date: </b>Wednesday, January 3, 2018 at 10:51 PM<br>
<b>To: </b>llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>><br>
<b>Subject: </b>Re: [PATCH] ldd::COFF: initalize ErrorHandler with CanExitEarly value<u></u><u></u></span></p>
</div><div><div class="h5">
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">I would also like to request that this is backported to the release_60 branch.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">On Tue, Jan 2, 2018 at 11:48 PM, Andrew Kelley <<a href="mailto:superjoe30@gmail.com" target="_blank">superjoe30@gmail.com</a>> wrote:<u></u><u></u></p>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<div>
<div>
<div>
<p class="MsoNormal">Previously, the COFF driver would call exit(1) from the<br>
ErrorHandler in the case of a link error, even if<u></u><u></u></p>
</div>
<p class="MsoNormal">CanExitEarly=false was specified. Now it initializes<u></u><u></u></p>
</div>
<p class="MsoNormal">the ErrorHandler in the same way that the ELF driver does.<u></u><u></u></p>
<div>
<div>
<div>
<div>
<p class="MsoNormal"><br>
diff --git a/lld/COFF/Driver.cpp b/lld/COFF/Driver.cpp<br>
index 0e7db7b6ae3..df1ce324055 100644<br>
--- a/lld/COFF/Driver.cpp<br>
+++ b/lld/COFF/Driver.cpp<br>
@@ -57,6 +57,7 @@ bool link(ArrayRef<const char *> Args, bool CanExitEarly, raw_ostream &Diag) {<br>
   errorHandler().<wbr>ErrorLimitExceededMsg =<br>
       "too many errors emitted, stopping now"<br>
       " (use /ERRORLIMIT:0 to see all errors)";<br>
+  errorHandler().ExitEarly = CanExitEarly;<br>
   Config = make<Configuration>();<br>
   Config->Argv = {Args.begin(), Args.end()};<br>
   Config->CanExitEarly = CanExitEarly;<u></u><u></u></p>
</div>
</div>
</div>
</div>
</div>
</blockquote>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div></div></div>
</div>

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