[PATCH] D25204: Register Calling Convention, Clang changes

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Sat Oct 8 07:57:41 PDT 2016


erichkeane added a comment.

Quick point i meant to post earlier  Couldn't change ExtInfo size.



================
Comment at: include/clang/AST/Type.h:1381
     /// regparm and the calling convention.
-    unsigned ExtInfo : 9;
+    unsigned ExtInfo : 10;
 
----------------
ABataev wrote:
> Erich, do you really need this? You don't increase number of required bits anymore, so this code must be restored
After much debugging, I realized that this does NOT store the enum from above, it stores the CallingConv from include/clang/Basic/Specifiers.h, which we actually HAVE put over 16.  I didn't see anything that we could lose from that enum to replace it however.


https://reviews.llvm.org/D25204





More information about the cfe-commits mailing list