<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Aug 4, 2014, at 9:57 PM, Craig Topper <<a href="mailto:craig.topper@gmail.com">craig.topper@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><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></blockquote><div><br></div><div>Sure.</div><div><br></div>The <span class="Apple-tab-span" style="white-space:pre"> </span>x86OperandSets disassembler table is small (thanks!) so even a 20% increase (5->6) only amounts to 1376 bytes.</div><div><br></div><div><div>The constant is not used directly in the asm matcher but the ConversionTable does become wider after the addition of valign. There is an increase of 1034 bytes there. The MatchTables do not become wider as a consequence of this change.</div><div><br></div><div>Adam</div><div><br></div><br><blockquote type="cite"><div class="gmail_extra"><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>
</blockquote></div><br></body></html>