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

Reply via email to