rjmccall added a comment.

In D55662#1335402 <https://reviews.llvm.org/D55662#1335402>, @ahatanak wrote:

> Sorry, please ignore my previous comment. I was looking at the wrong place.
>
> The following code reaches `Sema::BuildDecltypeType` without going through 
> `ActOnDecltypeExpression`:
>
>   template <typename T>
>   void overloaded_fn(T);
>   decltype(auto) v5 = &overloaded_fn<int>;
>
>
> `Sema::BuildDecltypeType` is called from `Sema::DeduceAutoType`, so calling 
> `CheckPlaceholderExpr ` there should fix the assert when the test case above 
> is compiled.


Okay.  You may need to push an unevaluated context when doing that.


Repository:
  rC Clang

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

https://reviews.llvm.org/D55662



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

Reply via email to