[PATCH] D97156: [ASTMatchers] Make Param functors variadic

Stephen Kelly via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 2 13:19:46 PST 2021


steveire added a comment.

In D97156#2593209 <https://reviews.llvm.org/D97156#2593209>, @njames93 wrote:

> Is it not nicer to name the class `PolymorphicMatcher` and do away with `WithParamN`, It may require partial specialization for the empty parameters case.
> But I think it would be worth it.

I'm not sure any partial specialization is needed?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D97156/new/

https://reviews.llvm.org/D97156



More information about the cfe-commits mailing list