[PATCH] D76038: PR45000: Use Sema::SetParamDefaultArgument in TransformLambdaExpr

Aaron Puchert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 13 17:57:32 PDT 2020


aaronpuchert added a comment.

In D76038#1978667 <https://reviews.llvm.org/D76038#1978667>, @mstorsjo wrote:

> Ping - any update on this one? Are you waiting on @rsmith for input on where to place a testcase?


No, I just didn't get around to writing the test yet, thanks for reminding me.

Although input from @rsmith would be appreciated. There are two ways to write the test: rely on the assertion to catch the type mismatch or inspect an AST dump and look for an `ImplicitCastExpr`. I'm tending towards the latter option, because it doesn't rely on assertions. (Which aren't always on.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76038





More information about the cfe-commits mailing list