diff --git a/clang/include/clang/Basic/DiagnosticASTKinds.td b/clang/include/clang/Basic/DiagnosticASTKinds.td index 031117f2c4137..c81d17ed64108 100644 --- a/clang/include/clang/Basic/DiagnosticASTKinds.td +++ b/clang/include/clang/Basic/DiagnosticASTKinds.td @@ -998,6 +998,16 @@ def warn_npot_ms_struct : Warning< "data types with sizes that aren't a power of two">, DefaultError, InGroup; +// -Wpadded-bitfield +def warn_padded_struct_bitfield : Warning< + "padding %select{struct|interface|class}0 %1 with %2 " + "%select{byte|bit}3%s2 to align %4">, + InGroup, DefaultIgnore; +def warn_padded_struct_anon_bitfield : Warning< + "padding %select{struct|interface|class}0 %1 with %2 " + "%select{byte|bit}3%s2 to align anonymous bit-field">, + InGroup, DefaultIgnore; + // -Wpadded, -Wpacked def warn_padded_struct_field : Warning< "padding %select{struct|interface|class}0 %1 with %2 " @@ -1005,7 +1015,7 @@ def warn_padded_struct_field : Warning< InGroup, DefaultIgnore; def warn_padded_struct_anon_field : Warning< "padding %select{struct|interface|class}0 %1 with %2 " - "%select{byte|bit}3%s2 to align anonymous bit-field">, + "%select{byte|bit}3%s2 to align anonymous field">, InGroup, DefaultIgnore; def warn_padded_struct_size : Warning< "padding size of %0 with %1 %select{byte|bit}2%s1 to alignment boundary">, diff --git a/clang/include/clang/Basic/DiagnosticGroups.td b/clang/include/clang/Basic/DiagnosticGroups.td index 9a8f3f03b39d1..bfda89945d635 100644 --- a/clang/include/clang/Basic/DiagnosticGroups.td +++ b/clang/include/clang/Basic/DiagnosticGroups.td @@ -586,7 +586,8 @@ def ExplicitInitializeCall : DiagGroup<"explicit-initialize-call">; def OrderedCompareFunctionPointers : DiagGroup<"ordered-compare-function-pointers">; def PackedNonPod : DiagGroup<"packed-non-pod">; def Packed : DiagGroup<"packed", [PackedNonPod]>; -def Padded : DiagGroup<"padded">; +def PaddedBitField : DiagGroup<"padded-bitfield">; +def Padded : DiagGroup<"padded", [PaddedBitField]>; def UnalignedAccess : DiagGroup<"unaligned-access">; def PessimizingMove : DiagGroup<"pessimizing-move">; diff --git a/clang/lib/AST/RecordLayoutBuilder.cpp b/clang/lib/AST/RecordLayoutBuilder.cpp index f1f2275da44dc..5d4f930fca50e 100644 --- a/clang/lib/AST/RecordLayoutBuilder.cpp +++ b/clang/lib/AST/RecordLayoutBuilder.cpp @@ -2297,19 +2297,22 @@ void ItaniumRecordLayoutBuilder::CheckFieldPadding( PadSize = PadSize / CharBitNum; InBits = false; } - if (D->getIdentifier()) - Diag(D->getLocation(), diag::warn_padded_struct_field) + if (D->getIdentifier()) { + auto Diagnostic = D->isBitField() ? diag::warn_padded_struct_bitfield + : diag::warn_padded_struct_field; + Diag(D->getLocation(), Diagnostic) << getPaddingDiagFromTagKind(D->getParent()->getTagKind()) - << Context.getTypeDeclType(D->getParent()) - << PadSize + << Context.getTypeDeclType(D->getParent()) << PadSize << (InBits ? 1 : 0) // (byte|bit) << D->getIdentifier(); - else - Diag(D->getLocation(), diag::warn_padded_struct_anon_field) + } else { + auto Diagnostic = D->isBitField() ? diag::warn_padded_struct_anon_bitfield + : diag::warn_padded_struct_anon_field; + Diag(D->getLocation(), Diagnostic) << getPaddingDiagFromTagKind(D->getParent()->getTagKind()) - << Context.getTypeDeclType(D->getParent()) - << PadSize + << Context.getTypeDeclType(D->getParent()) << PadSize << (InBits ? 1 : 0); // (byte|bit) + } } if (isPacked && Offset != UnpackedOffset) { HasPackedField = true; diff --git a/clang/test/CodeGenCXX/warn-all-padded-packed-packed-non-pod.cpp b/clang/test/CodeGenCXX/warn-all-padded-packed-packed-non-pod.cpp index 2a75498d87197..5e166deba0a3f 100644 --- a/clang/test/CodeGenCXX/warn-all-padded-packed-packed-non-pod.cpp +++ b/clang/test/CodeGenCXX/warn-all-padded-packed-packed-non-pod.cpp @@ -1,5 +1,5 @@ -// RUN: %clang_cc1 -triple=x86_64-none-none -Wpadded -Wpacked -verify=expected,top %s -emit-llvm-only -// RUN: %clang_cc1 -triple=x86_64-none-none -Wpadded -Wpacked -verify=expected,abi15 -fclang-abi-compat=15 %s -emit-llvm-only +// RUN: %clang_cc1 -triple=x86_64-none-none -Wpadded -Wpacked -Wno-padded-bitfield -verify=expected,top %s -emit-llvm-only +// RUN: %clang_cc1 -triple=x86_64-none-none -Wpadded -Wpacked -Wno-padded-bitfield -verify=expected,abi15 -fclang-abi-compat=15 %s -emit-llvm-only // -Wpacked-non-pod itself should not emit the "packed attribute is unnecessary" warnings. // RUN: %clang_cc1 -triple=x86_64-none-none -Wpacked-non-pod -verify=top %s -emit-llvm-only // -Wall should not emit the "packed attribute is unnecessary" warnings without -Wpacked. diff --git a/clang/test/CodeGenCXX/warn-padded-bitfields.cpp b/clang/test/CodeGenCXX/warn-padded-bitfields.cpp new file mode 100644 index 0000000000000..f9882d03564fe --- /dev/null +++ b/clang/test/CodeGenCXX/warn-padded-bitfields.cpp @@ -0,0 +1,41 @@ +// RUN: %clang_cc1 -triple=x86_64-none-none -Wpadded-bitfield -verify=expected %s -emit-llvm-only + +struct S1 { + unsigned a : 1; + unsigned long long : 0; // expected-warning {{padding struct 'S1' with 63 bits to align anonymous bit-field}} +}; + +struct S2 { + unsigned a : 1; + unsigned long long b : 64; // expected-warning {{padding struct 'S2' with 63 bits to align 'b'}} +}; + +struct S3 { + char a : 1; + short b : 16; // expected-warning {{padding struct 'S3' with 15 bits to align 'b'}} +}; + +struct [[gnu::packed]] S4 { + char a : 1; + short b : 16; +}; + +struct S5 { + unsigned a : 1; + unsigned long long b : 63; +}; + +struct S6 { + unsigned a : 1; + unsigned long long b; +}; + +struct S7 { + int word; + struct { + int filler __attribute__ ((aligned (8))); + }; +}; + +// The warnings are emitted when the layout of the structs is computed, so we have to use them. +void f(S1, S2, S3, S4, S5, S6, S7){}