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

Reply via email to