From f090048c44457c251865b9ed957513f1561a2e12 Mon Sep 17 00:00:00 2001 From: Richard Smith Date: Thu, 17 Nov 2016 02:16:09 +0000 Subject: [PATCH] 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 --- include/clang/Basic/DiagnosticGroups.td | 1 - include/clang/Basic/DiagnosticSemaKinds.td | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) 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">, -- 2.40.0