[PATCH] D50630: [AST] Update/correct the static_asserts for the bit-fields in Type

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 13 06:01:39 PDT 2018


erichkeane added inline comments.


================
Comment at: include/clang/AST/Type.h:1639
+
+    static_assert(sizeof(TypeBitfields) <= 4,
+                  "TypeBitfields is larger than 4 bytes!");
----------------
I don't really see value in ensuring that TypeBitfields is <= 4 bytes, it seems to me that all we care about is the union size, so just <=8 is fine.


Repository:
  rC Clang

https://reviews.llvm.org/D50630





More information about the cfe-commits mailing list