[PATCH] D105457: [clang] Refactor AST printing tests to share more infrastructure

[Douglas Yung via Phabricator via cfe-commits]

dyung added a comment.

In D105457#2878292 <https://reviews.llvm.org/D105457#2878292>, @nridge wrote:

> Making the default argument a non-lambda seems to be sufficient to avoid the error:
>
>   template <typename>
>   class function {
>   public:
>     template <typename F>
>     function(F) {}
>   };
>   
>   void DefaultFunc();
>   
>   template <typename M>
>   void Foo(M, function<void()> = DefaultFunc);
>   
>   void Bar() {
>     Foo(42);
>     Foo(42.0);
>   }

Interesting. If you can get me an updated patch, I can give it a try on my machine with 7.5 to verify if you like.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105457