<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 17, 2014 at 3:11 PM, Tom Stellard <span dir="ltr"><<a href="mailto:tom@stellard.net" target="_blank">tom@stellard.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Thu, Jul 17, 2014 at 04:55:00PM +0000, Chandler Carruth wrote:<br>
> Hi hfinkel, grosbach,<br>
><br>
> This allows the combiner to produce new nodes which need to go back<br>
> through legalization. This is particularly useful when generating<br>
> operands to target specific nodes in a post-legalize DAG combine where<br>
> the operands are significantly easier to express as pre-legalized<br>
> operations. My immediate use case will be PSHUFB formation where we need<br>
> to build a constant shuffle mask with a build_vector node.<br>
><br>
<br>
</div>Since the post-legalize DAG combiner is now doing combining and<br>
legalizing, do we still need to run legalize as a separate pass.<br>
Can we instead just do:<br>
<br>
combine -> legalize types -> combine w/ legalize ops</blockquote></div><br>I'm specifically going to try this, but wanted to get at least the meat of the patch out there to make sure folks were still OK with the general direction and the necessary changes I had to make to the legalization engine.</div>
</div>