<div dir="ltr">Hi Nadav,<div><br></div><div>I'll separate the patch in three commits:</div><div>- Actually lower VSELECTs, possibly with no additional functionality</div><div>  - Optimize lowered VSELECTs when possible</div>
<div>- PerformSELECTCombine changes</div><div><br></div><div>  Filipe</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, May 14, 2014 at 1:22 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">Overall this change makes sense. I did not review the patch carefully but I noticed that Andrea did. When you commit please split this patch to the different optimizations: shuffle blends, BLENDI, etc.<br>

<br>
</div><a href="http://reviews.llvm.org/D3752" target="_blank">http://reviews.llvm.org/D3752</a><br>
<br>
<br>
</blockquote></div><br></div>