[clang] [Sema] Add check for bitfield assignments to larger integral types (PR #68276)

via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 6 12:55:27 PDT 2023


================
@@ -6171,6 +6171,9 @@ def warn_signed_bitfield_enum_conversion : Warning<
   "signed bit-field %0 needs an extra bit to represent the largest positive "
   "enumerators of %1">,
   InGroup<BitFieldEnumConversion>, DefaultIgnore;
+def warn_bitfield_too_small_for_integral_type : Warning<
+  "bit-field %0 (%1 bits) is not wide enough to store type of %2 ">,
+  InGroup<Conversion>, DefaultIgnore;
----------------
vabridgers wrote:

Thanks @AaronBallman. I've tried the change I made on our internal code base, and see quite a few interesting findings we had not seen before. I will try the change on llvm/clang, and a few open source projects and report back. I'll also consider the gcc phrasing as a suggested improvement for the message. Best!

https://github.com/llvm/llvm-project/pull/68276


More information about the cfe-commits mailing list