<div dir="ltr"><div class="gmail_extra">This patch doesn't really make sense to me...</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 30, 2014 at 10:26 AM, Robert Khasanov <span dir="ltr"><<a href="mailto:rob.khasanov@gmail.com" target="_blank">rob.khasanov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":3tg" class="a3s" style="overflow:hidden">[x86] Simplify vector selection if condition value type matches vselect value type and true value is all ones or false value is all zeros.<br>
This transformation worked if selector is produced by SETCC, however SETCC is needed only if we consider to swap operands. So I replaced SETCC check for this case.<br></div></blockquote><div><br></div><div>What is your actual goal here?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":3tg" class="a3s" style="overflow:hidden">
Also I removed special case for cmp instructions in getVectorMaskingNode. Now cmp intrinsics lower as other intrinsics through VSELECT, and then VSELECT tranforms to AND in PerformSELECTCombine.<br></div></blockquote><div><br></div><div>This should be a separate patch.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":3tg" class="a3s" style="overflow:hidden">
New tests are not added, existed intrinsics tests on vpcmpeq{bwdq} cover these cases.</div></blockquote></div><br>I don't get it.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Does this patch have no functional impact? If so, say so, and justify both why it should have no functional impact and why the refactoring is desirable? The change looks like it *should* have functional impact...</div><div class="gmail_extra"><br></div><div class="gmail_extra">Also, you did see the recent churn on VSELECT? I'm planning to implement the new plan I outlined in my commit log. It would be great for you to review those patches and the plan I outlined and comment on it (I asked for comments).</div></div>