Support expression evaluation inside of actors. #2902
Conversation
| @@ -0,0 +1,14 @@ | |||
| actor Actor { | |||
| func f(_ n : Int) async { | |||
| f(0) // Force a coro-split. | |||
There was a problem hiding this comment.
Does this not require an await?
Why does this require a forced coroutine split?
There was a problem hiding this comment.
It doesn't. Originally I thought a .resume function would be more interesting, but there isn't really a difference for this case. I'll remove it.
| m_extension_decl = func_finder.m_extension_decl; | ||
| if (m_extension_decl) { | ||
| m_function_decl = func_finder.m_function_decl; | ||
| m_wrapper_decl = func_finder.m_wrapper_decl; | ||
| } else { | ||
| m_function_decl = func_finder.m_wrapper_decl; | ||
| m_wrapper_decl = nullptr; | ||
| } |
There was a problem hiding this comment.
to verify my reading, this and the above changes are NFC, correct?
There was a problem hiding this comment.
They correcting for the fact that now both lldb_expr and lldb_wrapped_expr have the @LLDBDebuggerFunction attribute. I don't like the implementation as is and I hope to update this with something better.
rdar://75905336
|
Updated patch is — I hope — much cleaner. |
|
test with swiftlang/swift#37155 |
|
@swift-ci test |
|
Closing, this has been superceded by #2904. |
rdar://75905336
This is a draft to help make sense of swiftlang/swift#37155. It needs some more work.