<div dir="ltr">Just curious how much does 6 operands increase the size of the static tables for the disassembler and the asm matcher?</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Aug 4, 2014 at 3:53 PM, Adam Nemet <span dir="ltr"><<a href="mailto:anemet@apple.com" target="_blank">anemet@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Apologies but the wrong version was sent out for the final patch in the series.  This should be the correct set now.<br>

<div class="HOEnZb"><div class="h5"><br>
On Aug 4, 2014, at 12:12 PM, Adam Nemet <<a href="mailto:anemet@apple.com">anemet@apple.com</a>> wrote:<br>
<br>
> Hi,<br>
><br>
> I marked this patch X86 rather AVX512 because while its main purpose is to enable masking with valignd/q it also changes a bit how PALIGNR is handled.  I also had to increase X86_MAX_OPERANDS from 5 to 6 which is also an X86-wide change.<br>

><br>
> The flow of the patchset is essentially to do some refactoring on the related TD part to make the rest of the patches simpler, then split the DAG nodes PALINGR and VALIGN, then add the masking support along with the intrinsics.<br>

><br>
> See the individual patch files for more detailed explanation.<br>
><br>
> Please let me know if it looks good.  The clang counterpart to add the intrinsic header piece is ready to go to the CFE list.<br>
><br>
> Adam<br>
><br>
</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>~Craig
</div>