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

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

Reply via email to