[PATCH] D157879: [clang] Report missing designated initializers in C++

Timm Bäder via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 17 21:29:45 PDT 2023


tbaeder added a comment.

In D157879#4595058 <https://reviews.llvm.org/D157879#4595058>, @Fznamznon wrote:

> In D157879#4594790 <https://reviews.llvm.org/D157879#4594790>, @tbaeder wrote:
>
>> Tangentially related: Now that we have this `InitializedFields` set, would it be easy to add a warning for double-initialization of fields (that would also trigger in C)?
>
> Isn't a warning like this already in place - https://godbolt.org/z/E1dGsY3ze ? Or you meant something else?

Yes, thanks. Someone else was asking for this feature a while back but looks like they didn't notice it already exists.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157879



More information about the cfe-commits mailing list