Author: Aaron Ballman Date: 2024-11-08T13:07:05-05:00 New Revision: 4661467003e7bc7f9bb89ab581517617d2a36c62
URL: https://github.com/llvm/llvm-project/commit/4661467003e7bc7f9bb89ab581517617d2a36c62 DIFF: https://github.com/llvm/llvm-project/commit/4661467003e7bc7f9bb89ab581517617d2a36c62.diff LOG: [C2y] Add test coverage and documentation for WG14 N3341 (#115478) This paper made empty structures and unions implementation-defined. We have always supported this as a GNU extension, so now we're documenting our behavior and removing the extension warning in C2y mode. Added: clang/test/C/C2y/n3341.c Modified: clang/docs/LanguageExtensions.rst clang/docs/ReleaseNotes.rst clang/lib/Sema/SemaDecl.cpp clang/www/c_status.html Removed: ################################################################################ diff --git a/clang/docs/LanguageExtensions.rst b/clang/docs/LanguageExtensions.rst index a051eb95898ecf..0998e6b30e229e 100644 --- a/clang/docs/LanguageExtensions.rst +++ b/clang/docs/LanguageExtensions.rst @@ -5975,6 +5975,17 @@ Clang guarantees the following behaviors: Currently, the above extension only applies to C source code, not C++. + +Empty Objects in C +================== +The declaration of a structure or union type which has no named members is +undefined behavior (C23 and earlier) or implementation-defined behavior (C2y). +Clang allows the declaration of a structure or union type with no named members +in all C language modes. `sizeof` for such a type returns `0`, which is + diff erent behavior than in C++ (where the size of such an object is typically +`1`). + + Qualified function types in C ============================= Declaring a function with a qualified type in C is undefined behavior (C23 and diff --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst index 77ba5f1d79bcdb..c3424e0e6f34c9 100644 --- a/clang/docs/ReleaseNotes.rst +++ b/clang/docs/ReleaseNotes.rst @@ -298,6 +298,12 @@ C2y Feature Support paper adopts Clang's existing practice, so there were no changes to compiler behavior. +- Implemented support for `N3341 <https://www.open-std.org/jtc1/sc22/wg14/www/docs/n3341.pdf>`_ + which makes empty structure and union objects implementation-defined in C. + ``-Wgnu-empty-struct`` will be emitted in C23 and earlier modes because the + behavior is a conforming GNU extension in those modes, but will no longer + have an effect in C2y mode. + - Updated conformance for `N3342 <https://www.open-std.org/jtc1/sc22/wg14/www/docs/n3342.pdf>`_ which made qualified function types implementation-defined rather than undefined. Clang has always accepted ``const`` and ``volatile`` qualified diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp index 6b0b4840a1eb2c..61c29e320d5c73 100644 --- a/clang/lib/Sema/SemaDecl.cpp +++ b/clang/lib/Sema/SemaDecl.cpp @@ -19365,11 +19365,12 @@ void Sema::ActOnFields(Scope *S, SourceLocation RecLoc, Decl *EnclosingDecl, } // Structs without named members are extension in C (C99 6.7.2.1p7), - // but are accepted by GCC. - if (NonBitFields == 0 && !getLangOpts().CPlusPlus) { - Diag(RecLoc, IsEmpty ? diag::ext_empty_struct_union : - diag::ext_no_named_members_in_struct_union) - << Record->isUnion(); + // but are accepted by GCC. In C2y, this became implementation-defined + // (C2y 6.7.3.2p10). + if (NonBitFields == 0 && !getLangOpts().CPlusPlus && !getLangOpts().C2y) { + Diag(RecLoc, IsEmpty ? diag::ext_empty_struct_union + : diag::ext_no_named_members_in_struct_union) + << Record->isUnion(); } } } else { diff --git a/clang/test/C/C2y/n3341.c b/clang/test/C/C2y/n3341.c new file mode 100644 index 00000000000000..523c3dd945ac1d --- /dev/null +++ b/clang/test/C/C2y/n3341.c @@ -0,0 +1,16 @@ +// RUN: %clang_cc1 -verify -std=c2y -Wall -pedantic %s +// RUN: %clang_cc1 -verify=gnu -Wall -pedantic %s + +/* WG14 N3341: Yes + * Slay Some Earthly Demons III + * + * Empty structure and union objects are now implementation-defined. + */ + +// expected-no-diagnostics + +struct R {}; // gnu-warning {{empty struct is a GNU extension}} +struct S { struct { }; }; // gnu-warning {{empty struct is a GNU extension}} +struct T { int : 0; }; // gnu-warning {{struct without named members is a GNU extension}} +union U {}; // gnu-warning {{empty union is a GNU extension}} + diff --git a/clang/www/c_status.html b/clang/www/c_status.html index 663cc2e33f5157..793e7006822cc1 100644 --- a/clang/www/c_status.html +++ b/clang/www/c_status.html @@ -191,7 +191,7 @@ <h2 id="c2y">C2y implementation status</h2> <tr> <td>Slay Some Earthly Demons III</td> <td><a href="https://www.open-std.org/jtc1/sc22/wg14/www/docs/n3341.pdf";>N3341</a></td> - <td class="unknown" align="center">Unknown</td> + <td class="full" align="center">Yes</td> </tr> <tr> <td>Slay Some Earthly Demons IV</td> _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
