[PATCH] D100057: Remove warning "suggest braces" for aggregate initialization of an empty class with an aggregate base class.

Hana Dusíková via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 7 12:02:39 PDT 2021


hankadusikova marked an inline comment as done.
hankadusikova added a comment.

In D100057#2674820 <https://reviews.llvm.org/D100057#2674820>, @lebedev.ri wrote:

> This patch says what it does, but not why it does what it does.
> Did the pattern become idiomatic so that we now want to not warn?

Yes, using inheritance for strong typing with aggregates is quite common pattern. I checked GCC / EDG / MSVC and neither of them is emitting warning in this case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100057



More information about the cfe-commits mailing list