<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Jan 4, 2014, at 9:38 AM, Alp Toker <<a href="mailto:alp@nuanti.com">alp@nuanti.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><br>On 04/01/2014 17:31, Renato Golin wrote:<br><blockquote type="cite">On 4 January 2014 17:06, Alp Toker <<a href="mailto:alp@nuanti.com">alp@nuanti.com</a> <<a href="mailto:alp@nuanti.com">mailto:alp@nuanti.com</a>>> wrote:<br><br> I wrote this patch (modulo the "METHOD" naming, sigh) and it's not<br> to blame.<br><br> The true culprit was r198478:<br><br> Fix PR18361: Invalidate LoopDispositions after LoopSimplify<br> hoists things.<br><br> getSCEV for an ashr instruction creates an intermediate zext<br> expression when it truncates its operand.<br><br> Reverted in r198503 until Andy gets a chance to look into it.<br><br><br>Hi Alp,<br><br>Thanks! The bot is running your commit now, should be fixed in a few minutes.<br><br>Let me know if you or Andy need help with an ARM device to test on.<br></blockquote><br>The crash reproduces on x86 BTW.<br><br>I'm not looking into a fix but feel free to re-land if you spot the issue before Andy's back.<br></div></blockquote></div><br><div>Thanks for taking care of this. I’ll follow up on it.</div><div>-Andy</div></body></html>