shafik added a comment.

LGTM, I wlll let @royjacobson give the final approval after reviewing codegen 
tests.



================
Comment at: clang/lib/Sema/SemaDeclCXX.cpp:9161-9162
+    if (CSM == Sema::CXXDefaultConstructor) {
+      // Default constructor of a union can also be valid if one of the
+      // members has an explicit initializer.
+      const auto *RD = cast<CXXRecordDecl>(Field->getParent());
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145851

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to