Skip to content

Cherry-pick coro async mangling and single predecessor phi fix #3074

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

aschwaighofer
Copy link

No description provided.

@aschwaighofer
Copy link
Author

@swift-ci Please test

1 similar comment
@aschwaighofer
Copy link
Author

@swift-ci Please test

@aschwaighofer aschwaighofer force-pushed the coro_async_cherry_picks_mangling_single_pred_phis branch from 76ddba0 to 6bca7ff Compare July 7, 2021 15:59
@aschwaighofer
Copy link
Author

@swift-ci test

The resume partial functions generated for swift suspend points will now
use a Swift mangling suffix.

Await resume partial functions will use the suffix 'TQ'[0-9]+'_' (e.g "...TQ0_")
and suspend resume partial functions will use the suffix 'TY'[0-9]+'_'
(e.g "...TY1_").

Reviewed By: nate_chandler

Differential Revision: https://reviews.llvm.org/D104144
Code assumes that uses of single predecessor phis are not live accross
suspend points. Cleanup any single predecessor phis preceeding the code
making this assumption.

rdar://76020301

Differential Revision: https://reviews.llvm.org/D105488
While upstreaming patches this code somehow was applied to the wrong switch statement.

Differential Revision: https://reviews.llvm.org/D105504
@aschwaighofer aschwaighofer force-pushed the coro_async_cherry_picks_mangling_single_pred_phis branch from 6bca7ff to 7438d49 Compare July 7, 2021 18:08
@aschwaighofer
Copy link
Author

@swift-ci test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants