From: Richard Smith Date: Thu, 17 Nov 2016 02:16:09 +0000 (+0000) Subject: Remove -Wsigned-enum-bitfield from -Wmost. On a wide set of ABIs, this warning X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=f090048c44457c251865b9ed957513f1561a2e12;p=clang Remove -Wsigned-enum-bitfield from -Wmost. On a wide set of ABIs, this warning is completely irrelevant, producing (effectively) false positives, and -Wmost is used pretty widely. We should somehow turn it back on by default when targeting the MS ABI, however, since it indicates the program will not do as intended in those cases. (Or perhaps we should just treat enum bitfields as having the signedness of the enum, even when targeting the MS ABI...) git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@287193 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/clang/Basic/DiagnosticGroups.td b/include/clang/Basic/DiagnosticGroups.td index ad92b4da68..b3968dabeb 100644 --- a/include/clang/Basic/DiagnosticGroups.td +++ b/include/clang/Basic/DiagnosticGroups.td @@ -662,7 +662,6 @@ def Most : DiagGroup<"most", [ ReturnType, SelfAssignment, SelfMove, - SignedEnumBitfield, SizeofArrayArgument, SizeofArrayDecay, StringPlusInt, diff --git a/include/clang/Basic/DiagnosticSemaKinds.td b/include/clang/Basic/DiagnosticSemaKinds.td index 32c9ca03ee..316cbe2e7d 100644 --- a/include/clang/Basic/DiagnosticSemaKinds.td +++ b/include/clang/Basic/DiagnosticSemaKinds.td @@ -3023,7 +3023,7 @@ def warn_int_to_void_pointer_cast : Warning< def warn_no_underlying_type_specified_for_enum_bitfield : Warning< "enums in the Microsoft ABI are signed integers by default; consider giving " "the enum %0 an unsigned underlying type to make this code portable">, - InGroup>, DefaultIgnore; + InGroup, DefaultIgnore; def warn_attribute_packed_for_bitfield : Warning< "'packed' attribute was ignored on bit-fields with single-byte alignment " "in older versions of GCC and Clang">,