aaron.ballman added inline comments.
================ Comment at: clang/include/clang/AST/DeclCXX.h:1430 + /// out of which one will be selected at the end. + /// This is called separately from addedMember because it has to be deferred + /// to the completion of the class. ---------------- erichkeane wrote: > royjacobson wrote: > > erichkeane wrote: > > > @aaron.ballman is messing around with addedMember recently... I wonder > > > if there are OTHER decisions that need to be updated/moved here? > > Do you know what papers/DRs he's working on? It seemed like most constexpr > > stuff is done per-function and not per-class. > > > > I don't think there's something else that needs to change but it's > > definitely possible I missed something.. I went through addedMember and > > tried to remove everything destructor related basically. > > > > > He's working on DR647, which does a lot of work with special member > functions/constexpr/consteval constructors. > > That said, I believe he said it isn't a problem, they are related, but not > conflicting in a way that he had. > > I don't see anything here that should conflict with what I'm working on (this is mostly about destructors and I've not started touching those yet). I'm working specifically on implementing this bit: http://eel.is/c++draft/dcl.constexpr#7 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126194/new/ https://reviews.llvm.org/D126194 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits