<div dir="ltr"><div>Working to reverse the patch. I need someone with MSVC to help me.<br><br></div>-- HT<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 24, 2016 at 3:19 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron.ballman@gmail.com" target="_blank">aaron.ballman@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, Jun 24, 2016 at 8:16 AM, Hubert Tong<br>
<span class=""><<a href="mailto:hubert.reinterpretcast@gmail.com">hubert.reinterpretcast@gmail.com</a>> wrote:<br>
> On Fri, Jun 24, 2016 at 3:07 PM, Aaron Ballman <<a href="mailto:aaron.ballman@gmail.com">aaron.ballman@gmail.com</a>><br>
> wrote:<br>
>><br>
>> On Fri, Jun 24, 2016 at 8:05 AM, Hubert Tong<br>
>> <<a href="mailto:hubert.reinterpretcast@gmail.com">hubert.reinterpretcast@gmail.com</a>> wrote:<br>
>> > On Fri, Jun 24, 2016 at 3:00 PM, Aaron Ballman <<a href="mailto:aaron.ballman@gmail.com">aaron.ballman@gmail.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> aaron.ballman added a comment.<br>
>> >><br>
>> >> Did Richard sign off on this off-line? Also, this has caused a bot<br>
>> >> failure:<br>
>> ><br>
>> > That was my understanding.<br>
>><br>
>> Ah, I meant, "I approve this, but wait for Richard to explicitly<br>
>> approve before committing." I'm sorry if I didn't make that clear<br>
>> enough! If Richard okayed it offline with you, that's fine (but he<br>
>> should also sign off in phab, or post a comment) so everyone following<br>
>> the thread knows.<br>
><br>
> I meant that my understanding is that Richard has okayed it offline.<br>
<br>
</span>Thank you for the clarification.<br>
<span class=""><br>
> As for the bot failure, it appears it applies local patches to avoid<br>
> compiler bugs.<br>
> I am not sure what the policy is if a bot failure has more to do with the<br>
> bot set-up than the change (assuming that is the case here).<br>
> In any case, I am not sure I can resolve the failure without cooperation<br>
> from the bot owner.<br>
<br>
</span>More bots have gone red in the meantime:<br>
<br>
8:07 AM <llvmbb> build #7517 of<br>
llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast is complete: Failure<br>
[failed build]  Build details are at<br>
<a href="http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast/builds/7517" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast/builds/7517</a><br>
 blamelist: grimar, hubert.reinterpretcast<br>
8:12 AM <llvmbb> build #13373 of clang-x64-ninja-win7 is complete:<br>
Failure [failed build stage 1]  Build details are at<br>
<a href="http://lab.llvm.org:8011/builders/clang-x64-ninja-win7/builds/13373" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/clang-x64-ninja-win7/builds/13373</a><br>
blamelist: hubert.reinterpretcast<br>
<br>
Are these also failing due to local patches on the bots?<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
> -- HT<br>
><br>
>><br>
>><br>
>> ~Aaron<br>
>><br>
>> ><br>
>> >><br>
>> >><br>
>> >> 7:57 AM <bb-chapuni> build #4390 of ninja-clang-i686-msc19-R is<br>
>> >> complete:<br>
>> >> Failure [failed build_clang_tools_1]  Build details are at<br>
>> >> <a href="http://bb.pgr.jp/builders/ninja-clang-i686-msc19-R/builds/4390" rel="noreferrer" target="_blank">http://bb.pgr.jp/builders/ninja-clang-i686-msc19-R/builds/4390</a><br>
>> >> blamelist:<br>
>> >> Hubert Tong <<a href="mailto:hubert.reinterpretcast@gmail.com">hubert.reinterpretcast@gmail.com</a>><br>
>> >><br>
>> >><br>
>> >> <a href="http://reviews.llvm.org/D19770" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19770</a><br>
>> >><br>
>> >><br>
>> >><br>
>> ><br>
><br>
><br>
</div></div></blockquote></div><br></div>