
<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Nov 17, 2010, at 1:12 PM, Anton Korobeynikov wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>Hi Evan,<br><br><blockquote type="cite">--- llvm/trunk/test/CodeGen/ARM/select_xform.ll (original)<br></blockquote><blockquote type="cite">+++ llvm/trunk/test/CodeGen/ARM/select_xform.ll Wed Nov 17 14:13:28 2010<br></blockquote><blockquote type="cite">@@ -8,7 +8,8 @@<br></blockquote><blockquote type="cite"> ; ARM: movgt r0, r1<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite"> ; T2: t1:<br></blockquote><blockquote type="cite">-; T2: sub.w r0, r1, #-2147483648<br></blockquote><blockquote type="cite">+; T2: mvn r0, #-2147483648<br></blockquote><blockquote type="cite">+; T2: add r0, r1<br></blockquote>Isn't this a regression?<br></div></blockquote><div><br></div>It's not. Before the patch it uses two sub instructions, with the first one being the .w variant. The test just didn't look for it.</div><div><br></div><div>Evan</div><div><br><blockquote type="cite"><div><br>-- <br>With best regards, Anton Korobeynikov<br>Faculty of Mathematics and Mechanics, Saint Petersburg State University<br></div></blockquote></div><br></body></html>