<div dir="ltr"><div><div><div>Hans,<br><br></div>I resubmitted this as a Phabricator patch, and Rui gave it approval for 6.0.<br><br></div>Regards,<br></div>Andrew<br><div><div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 16, 2018 at 9:54 AM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Looks fine for 6.0 from my point of view.<br>
<br>
Rui, what do you think?<br>
<div class="HOEnZb"><div class="h5"><br>
On Thu, Jan 4, 2018 at 7:37 PM, Andrew Kelley <<a href="mailto:superjoe30@gmail.com">superjoe30@gmail.com</a>> wrote:<br>
><br>
><br>
> On Thu, Jan 4, 2018 at 2:37 AM, Shoaib Meenai <<a href="mailto:smeenai@fb.com">smeenai@fb.com</a>> wrote:<br>
>><br>
>> +Rui and Hans, in case they miss it on the list.<br>
>><br>
>><br>
>><br>
>> (Side note: it's much easier to keep track of patches on Phabricator IMO.)<br>
><br>
><br>
> Thank you Shoaib, I will try that next time.<br>
><br>
>><br>
>><br>
>><br>
>> From: llvm-commits <<a href="mailto:llvm-commits-bounces@lists.llvm.org">llvm-commits-bounces@lists.<wbr>llvm.org</a>> on behalf of<br>
>> Andrew Kelley via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>><br>
>> Reply-To: Andrew Kelley <<a href="mailto:superjoe30@gmail.com">superjoe30@gmail.com</a>><br>
>> Date: Wednesday, January 3, 2018 at 10:51 PM<br>
>> To: llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>><br>
>> Subject: Re: [PATCH] ldd::COFF: initalize ErrorHandler with CanExitEarly<br>
>> value<br>
>><br>
>><br>
>><br>
>> I would also like to request that this is backported to the release_60<br>
>> branch.<br>
>><br>
>><br>
>><br>
>> On Tue, Jan 2, 2018 at 11:48 PM, Andrew Kelley <<a href="mailto:superjoe30@gmail.com">superjoe30@gmail.com</a>><br>
>> wrote:<br>
>><br>
>> Previously, the COFF driver would call exit(1) from the<br>
>> ErrorHandler in the case of a link error, even if<br>
>><br>
>> CanExitEarly=false was specified. Now it initializes<br>
>><br>
>> the ErrorHandler in the same way that the ELF driver does.<br>
>><br>
>><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<br>
>> 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;<br>
>><br>
>><br>
><br>
><br>
</div></div></blockquote></div><br></div></div></div></div></div></div>