erichkeane added inline comments.
================ Comment at: clang/lib/Sema/SemaConcept.cpp:1079 MultiLevelTemplateArgumentList MLTAL = S.getTemplateInstantiationArgs( - CSE->getNamedConcept(), /*Final=*/false, &TAL, + CSE->getNamedConcept(), /*Final=*/true, &TAL, /*RelativeToPrimary=*/true, ---------------- mizvekov wrote: > erichkeane wrote: > > Comment here should be "RelativeToPrimary=" I think. > This is a new parameter introduced in D134604, you would have to read that > dependency first to figure that out :) > I forgot to mention that on the description of this patch. Ah! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136566/new/ https://reviews.llvm.org/D136566 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits