<div dir="ltr"><div>There is no new failure in the tests in Debug version on MSVC 2015, however a bunch of failures (~30) were masked on Release builds (the Test Methods count went from 769 to 1188). All the failures were already presents in the Debug builds (where the optimizer didn't mess up with the code due to LLVM_UNREACHERABLE). So I suppose it's still ok?<br></div><div><br></div>(BTW, I don't have commit rights, so I will depend on someone to commit it)<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 21, 2016 at 8:11 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">zturner accepted this revision.<br>
zturner added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
LGTM as long as you've run the test suite and confirmed everything works.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D25681" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D25681</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>