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