<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;"><div><br><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>-Tom<br><br><blockquote type="cite">Thanks for the test case.<br>-Quentin<br><br>On Feb 12, 2014, at 12:58 PM, Quentin Colombet <<a href="mailto:qcolombet@apple.com">qcolombet@apple.com</a>> wrote:<br><br><blockquote type="cite">Sure, I’m looking into it.<br><br>-Quentin<br><br>On Feb 12, 2014, at 12:03 PM, Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:<br><br><blockquote type="cite">Hi Quentin,<br><br>I've discovered a regression with this commit, please see the attached test case.<br>In this case, CodeGenPrepare is promoting the mul in a sext + mul pattern even<br>though the addressing mode it is creating isn't legal.<br><br>One interesting thing about this test case is that if you remove the<br>nsw from the mul instruction, then the incorrect transform does not<br>take place. I'm not sure why this matters. Would you mind taking a look?<br><br>Thanks,<br>Tom<br><codegen-prepare-addrmode-sext.ll><br></blockquote><br>_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</blockquote></blockquote></div></blockquote></div><br></body></html>