[cfe-dev] X86 intrinsics running out of IdentifierInfo Builtin IDs

[Benyei, Guy]

Hi all,
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.

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.

Please let me know if anyone is opposed to adding this additional bit, or if there are any other concerns/suggestions on this issue.

Thanks
[email_signature_guy_new2]

---------------------------------------------------------------------
Intel Israel (74) Limited

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20150701/a7683021/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: image001.png
Type: image/png
Size: 24800 bytes
Desc: image001.png
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20150701/a7683021/attachment.png>