<div dir="ltr"><div><div><div><div><div><div><div><div>I just wanted to float this patch out for some feedback - It's not ready for commit.<br></div><div>There is no functionality change - so no tests included.<br></div>
<div><br></div>This patch addresses the following:<br></div>  1) It replaces the use of the "__invoke" magic name for the static invoker, with a property within<br></div>    the lambda class  - I thought this would be a less brittle approach and potentially more efficient <br>
-   though it takes up more space in the LambdaDefinitionData - how big a deal is that?<br></div><div>      - also with my current change can i just drop the name for this function or do i need the<br></div><div>        "super-secret" name? <br>
</div><div><br><br></div>  2) Also I intend to put some utility functions in Lambda.h (I have included some samples - these functions currently might overlap in functionality - in my initial generic lambda implementation they were scattered amongst various files, and i thought it might be better to centralize them a little) - here are my questions:<br>
</div>     - do you prefer to place utility functions within CXXMethodDecl or Sema (ugh - the recompiles  <br>        suck with changes!) if they make sense there?<br></div></div>        That is do you prefer IsLambdaCallOperator(MD) or MD->IsLambdaCallOperator() ?<br>
<br></div>Thanks!<br clear="all"><div><div><div><div><div><div><div><div><div><div><div><div>Faisal Vali<br><br></div>
</div></div></div></div></div></div></div></div></div></div></div></div>