<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 28, 2014 at 11:04 AM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.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=""><div class="gmail_quote">On Thu, Aug 28, 2014 at 10:49 AM, Owen Anderson <span dir="ltr"><<a href="mailto:resistor@mac.com" target="_blank">resistor@mac.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="overflow:hidden">Do not introduce new shuffle patterns after operation legalization if SHUFFLE_VECTOR<br>

was marked custom.  The target independent DAG combine has no way to know if<br>
the shuffles it is introducing are ones that the target could support or not.</div></blockquote></div><br></div>Test case? This seems somewhat arbitrary otherwise...</blockquote></div><br>(To be completely clear, in case this email wasn't, I understand that currently not all backends support a custom lowering of all shuffles.... but the x86 backend actually does support it (for legal types) and so it seems quite likely that someone could accidentally change this back unless we have more checking in the backend to back up the assertion that we can't form an arbitrary shuffle here. Essentially, I'm fine with the patch, but I'm worried that I or someone else could easily re-break this in the future unless we have some test coverage.)</div>
</div>