[PATCH] D143109: [Sema] Push a LambdaScopeInfo before calling SubstDefaultArgument

Corentin Jabot via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 1 14:19:27 PST 2023


cor3ntin added a comment.

In D143109#4097775 <https://reviews.llvm.org/D143109#4097775>, @erichkeane wrote:

> @cor3ntin : Mind taking a look here?  You're my lambda expert these days :)

I did something similar to check the requires clause of a lambda and it does technically fix the issue, i don't see a more elegant solution, but it does sound a bit wrong... (it allocates a dummy function scope that is then kept around forever)
Maybe we can add a fixme comment here and add a "remove lambda scope" with an raii object for these two cases in a separate PR?

The issue is not surprising though, there are weird order of operations in how parameters and functions and their context are handled.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143109



More information about the cfe-commits mailing list