[clang] [Clang][Sema] Refactor collection of multi-level template argument lists (PR #106585)

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 17 10:06:17 PDT 2024


================
@@ -11392,9 +11392,9 @@ class Sema final : public SemaBase {
       CXXScopeSpec &SS, IdentifierInfo *Name, SourceLocation NameLoc,
       const ParsedAttributesView &Attr, TemplateParameterList *TemplateParams,
       AccessSpecifier AS, SourceLocation ModulePrivateLoc,
-      SourceLocation FriendLoc, unsigned NumOuterTemplateParamLists,
-      TemplateParameterList **OuterTemplateParamLists,
-      SkipBodyInfo *SkipBody = nullptr);
+      SourceLocation FriendLoc,
+      ArrayRef<TemplateParameterList *> OuterTemplateParamLists,
+      bool IsMemberSpecialization, SkipBodyInfo *SkipBody = nullptr);
----------------
erichkeane wrote:

I have a preference for using an enum instead of bool params (so that callers do: `SpecializationKind::None`, or `SpecializationKind::Member` or something like that?).  Even if there are two options, it improves readability of callers a ton.

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


More information about the cfe-commits mailing list