vitalybuka added a comment.

In D132275#3740479 <https://reviews.llvm.org/D132275#3740479>, @efriedma wrote:

> Among other things, we could use a definition from a different module.

Is this a thing, I assumed lambda is always defined in the module?

> Either we relax the requirements globally for all lambdas,

This looks like unnecessary special case.

> or we fix EmitLambdaDelegatingInvokeBody() so it passes a pointer to a proper 
> alloca instead of undef.

I will try to do this one, I assume we need any alloca and let optimized to 
eliminate it when it can.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132275

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

Reply via email to