erichkeane added a comment.

In D126907#3636830 <https://reviews.llvm.org/D126907#3636830>, @erichkeane 
wrote:

> SO I've been playing with this for a while and all the libcxx issues. I THINK 
> we really need to just bit the bullet and figure out how to correctly re-add 
> things to the instantiation scope (after making the CheckFunctionConstraints 
> LocalInstantiationScope 'false' for the 2nd param).  I can do it well enough 
> with functions since we have a function for it, but likely need to do the 
> same for at least RecordDecl.

URGH.... And its not clear to me that I CAN do this, since a containing 
function template's body doesn't exist yet, so we cant go back and make those 
work.  So we are at a point where we both have to inherit AND not inherit the 
paretn scope, and I have no idea on which basis to do so.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126907

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

Reply via email to