<div dir="ltr">It seems increasingly true that we care less about optimizing for size on 32-bit architectures. On a 64-bit platform, we had 32-bits of padding there already, so we can probably afford the extra space on 32-bit.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 1, 2015 at 6:12 AM, Benyei, Guy <span dir="ltr"><<a href="mailto:guy.benyei@intel.com" target="_blank">guy.benyei@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="EN-US" link="#0563C1" vlink="#954F72">
<div>
<p class="MsoNormal">Hi all,<u></u><u></u></p>
<p class="MsoNormal">My team is working on adding some more X86 intrinsics to Clang/LLVM, and we are hitting the limit of the number of builtins supported by Clang. The “ObjCOrBuiltinID” field in the IdentifierInfo class is 11 bits wide, which is not sufficient
 to represent all the new intrinsics, and we are getting the "ID too large for field!" assertion.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">The easiest fix seems to be to increase the size of this “ObjCOrBuiltinID” field to 12 bits, but that will make the bitfield in IdentifierInfo 33 bits wide, consuming an additional 32 bits in some architectures for each instance. A quick
 check shows, that this fix works without any functional regressions, but we have no performance numbers yet.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Please let me know if anyone is opposed to adding this additional bit, or if there are any other concerns/suggestions on this issue.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Thanks<u></u><u></u></p>
<p class="MsoNormal"><img width="420" height="116" src="cid:image001.png@01D0B416.42F9ED20" alt="email_signature_guy_new2"><u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<p>---------------------------------------------------------------------<br>
Intel Israel (74) Limited</p>

<p>This e-mail and any attachments may contain confidential material for<br>
the sole use of the intended recipient(s). Any review or distribution<br>
by others is strictly prohibited. If you are not the intended<br>
recipient, please contact the sender and delete all copies.</p></div>

<br>_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
<br></blockquote></div><br></div>