[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