[PATCH] D137872: Implement lambdas with inalloca parameters by forwarding to function without inalloca calling convention.

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 8 12:50:54 PDT 2023


efriedma added a comment.

I'm having a bit of trouble following how exactly the thunk creation is working here... do we generate different code depending on whether the call operator and/or the static invoker are referenced?  Why is the function in EmitLambdaInAllocaCallOpFn not getting defined using the normal CodeGenModule machinery?



================
Comment at: clang/lib/CodeGen/CGCall.cpp:767
+    (opts & FnInfoOpts::IsDelegateCall) == FnInfoOpts::IsDelegateCall;
+  CGFunctionInfo::Profile(ID, isInstanceMethod, isChainCall, info, paramInfos,
                           required, resultType, argTypes);
----------------
I think you need to pass isDelegateCall to CGFunctionInfo::Profile.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137872



More information about the cfe-commits mailing list