<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 Feb 12, 2014, at 4:08 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 12, 2014 at 4:05 PM, Nadav Rotem <span dir="ltr"><<a href="mailto:nrotem@apple.com" target="_blank">nrotem@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto;"><div class="">On Feb 12, 2014, at 3:54 PM, Owen Anderson <<a href="mailto:resistor@mac.com">resistor@mac.com</a>> wrote:<br>
<br>
> Author: resistor<br>
> Date: Wed Feb 12 17:54:07 2014<br>
> New Revision: 201275<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=201275&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=201275&view=rev</a><br>
> Log:<br>
> Remove a very old instcombine where<br>
<br>
</div>For vectors this optimization is always a win. Can you please bring it back and add a check that the comparison type is a vector?</blockquote></div><br>Do you have test cases which regressed? I find it odd that I saw no regressions from any of the heavily auto-vectorized benchmarks on x86-64.</div>
</div>
</blockquote></div><br><div>I don’t have a specific testcase that regressed, this is purely theoretical. </div></body></html>