[llvm-branch-commits] [clang] [clang-tools-extra] [clang] NFCI: use TemplateArgumentLoc for type-param DefaultArgument (PR #92854)

Matheus Izvekov via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Mon May 20 23:29:07 PDT 2024


================
@@ -10082,7 +10082,9 @@ class Sema final : public SemaBase {
 
   bool SubstTemplateArgument(const TemplateArgumentLoc &Input,
                              const MultiLevelTemplateArgumentList &TemplateArgs,
-                             TemplateArgumentLoc &Output);
+                             TemplateArgumentLoc &Output,
+                             SourceLocation Loc = {},
+                             const DeclarationName &Entity = {});
----------------
mizvekov wrote:

There are a few places we used SubsType, because we only had a type, now we use SubstTemplateArgument.

SubstTemplateArgument was missing arguments for setting Instantiation location and entity names.
Adding those is needed so we don't regress in diagnostics.

https://github.com/llvm/llvm-project/pull/92854


More information about the llvm-branch-commits mailing list