[PATCH] D126194: [Concepts] Implement overload resolution for destructors (P0848)

Fri Jun 17 06:38:19 PDT 2022

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