[PATCH] D131858: [clang] Track the templated entity in type substitution.
    Matheus Izvekov via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Thu Sep  1 06:33:54 PDT 2022
    
    
  
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
    
    
More information about the cfe-commits
mailing list