[PATCH] D64380: Add 'require_designated_init' and 'required' attribute to clang

Emmett Neyman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 11 16:16:20 PDT 2019


emmettneyman added inline comments.


================
Comment at: clang/include/clang/Basic/AttrDocs.td:1448
 
+def RequireDesignatedInitDocs : Documentation {
+  let Category = DocCatType;
----------------
aaron.ballman wrote:
> The behavior should be documented as to what happens when you apply these attributes to unions.
For now, adding a `requires_init` attribute to a field of a union will result in a warning that the attribute is ignored. In the future, I might submit another patch in the future adding behavior for this attribute for fields of a union.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64380/new/

https://reviews.llvm.org/D64380





More information about the cfe-commits mailing list