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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to