ahatanak added a comment.

In D55662#1335773 <https://reviews.llvm.org/D55662#1335773>, @rjmccall wrote:

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


Since I'm just moving the call to `CheckPlaceholderExpr` to the call site, I 
don't think I have to push an unevaluated context there?

Also, it looks like I can just change the check 
`Init->getType()->isNonOverloadPlaceholderType()`  at the beginning of 
`Sema::DeduceAutoType` to `Init->getType()->getAsPlaceholderType()` instead of 
inserting the call to `CheckPlaceholderExpr` right before the call to 
`BuildDecltypeType`.


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