<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text --><style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: #800000 2px solid; } --></style>
</head>
<body>
<meta content="text/html; charset=UTF-8">
<style type="text/css" style="">
<!--
p
{margin-top:0;
margin-bottom:0}
-->
</style>
<div dir="ltr">
<div id="x_divtagdefaultwrapper" style="font-size:12pt; color:#000000; background-color:#FFFFFF; font-family:Calibri,Arial,Helvetica,sans-serif">
<p>Thank you Renato. I will have a look at the failing tests first. I will chase you on IRC when I get my feet wet if that's fine.</p>
<p><br>
</p>
<p>Cheers,</p>
<p>Pablo<br>
</p>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> Renato Golin <renato.golin@linaro.org><br>
<b>Sent:</b> 08 September 2016 13:15:50<br>
<b>To:</b> Pablo Barrio<br>
<b>Cc:</b> LLVM Commits<br>
<b>Subject:</b> Re: [llvm] r280918 - Revert "[ARM] Lower UDIV+UREM to UDIV+MLS (and the same for SREM)"</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">On 8 September 2016 at 11:05, Pablo Barrio via llvm-commits<br>
<llvm-commits@lists.llvm.org> wrote:<br>
> Author: pabbar01<br>
> Date: Thu Sep 8 05:05:57 2016<br>
> New Revision: 280918<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=280918&view=rev">http://llvm.org/viewvc/llvm-project?rev=280918&view=rev</a><br>
> Log:<br>
> Revert "[ARM] Lower UDIV+UREM to UDIV+MLS (and the same for SREM)"<br>
><br>
> This reverts commit r280808.<br>
><br>
> It is possible that this change results in an infinite loop. This<br>
> is causing timeouts in some tests on ARM, and a Chromebook bot is<br>
> failing.<br>
<br>
Hi Pablo,<br>
<br>
It is possible, but I was not able to get to the bottom of this, as my<br>
Chromebook was crashing even before that patch. Though, I had an<br>
"experiment" with my Chromebook, so I wasn't very sure and started<br>
looking around it... :)<br>
<br>
If the test-suite passes, this could also fix both self-hosting bots,<br>
too. I also may have an idea why this is happening (validation loop).<br>
Ping me later, if you want to discuss this, but essentially, just<br>
testing on the test-suite shall point you in the right direction.<br>
<br>
cheers,<br>
--renato<br>
<br>
</div>
</span></font>IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use
it for any purpose, or store or copy the information in any medium. Thank you.
</body>
</html>