<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 1, 2014 at 2:37 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 dir="ltr">Hi Chandler,<div><br></div><div>In addition to Adam's comment (Thanks Adam for answers), you can see thread <a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20140929/237328.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20140929/237328.html</a></div><div><br></div><div>If you feel strongly, I can split this patch into two: VSELECT simplification (with test) and changes in intrinsics lowering.</div></div></blockquote></div><br>His comment makes sense.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Please add a test case specifically for the VSELECT simplification you are expecting that just uses VSELECT. We really don't want to build up reliance on DAG combines of generic DAG nodes that are only tested indirectly.</div></div>