mizvekov added a comment. In D131858#3763851 <https://reviews.llvm.org/D131858#3763851>, @erichkeane wrote:
> Just did a quick scroll through this (as it is quite large!), but the general > idea seems like a fine one to me. I AM concerned about how it interacts with > the deferred concepts instantiation that I've been working on > (https://reviews.llvm.org/D126907), particularly around the MLTAL work. I think I did rebase it on top of that at one point, thought it was short lived as it was reverted if I am not mistaken. But I can certainly do it again if you merge yours first, and I am available any time to help if it happens the other way around. But the gist of the change is simple, you will simply have to pass in the templated declaration for every level that you push into the MLTAL. I think that should be the only change that affects you, besides possibly churn in AST tests if you plan to have any. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131858/new/ https://reviews.llvm.org/D131858 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
