[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