[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