Skip to content
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

[wasm] Repair IRGen/async.swift test #71754

Conversation

kateinoigakukun
Copy link
Member

The test has been crashed with WebAssembly target due to lack of tail-call support guard in the CoroSplit llvm pass. The crash has been fixed by llvm/llvm-project@8c5c4d9 but the new codegen produces slightly different IR (call instead of tail call, while they are lowered to the same wasm instruction). So this patch adds new checks for wasm target.

The test has been crashed with WebAssembly target due to lack of tail-call
support guard in the CoroSplit llvm pass. The crash has been fixed by
llvm/llvm-project@8c5c4d9
but the new codegen produces slightly different IR (`call` instead of
`tail call`, while they are lowered to the same wasm instruction). So
this patch adds new checks for wasm target.
@kateinoigakukun
Copy link
Member Author

preset=buildbot_linux_crosscompile_wasm
@swift-ci Please test with preset Linux Platform

@kateinoigakukun
Copy link
Member Author

@swift-ci Please smoke test

@kateinoigakukun kateinoigakukun marked this pull request as ready for review February 21, 2024 05:28
@kateinoigakukun kateinoigakukun merged commit e1934c9 into swiftlang:main Feb 21, 2024
3 of 4 checks passed
@kateinoigakukun kateinoigakukun deleted the pr-e2b725c98aa1ec0fb01b5e6da0ee4fcc3d327c44 branch February 21, 2024 07:48
@kateinoigakukun
Copy link
Member Author

The failure on wasm CI was another unrelated issue and it's fixed by #71772

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.

1 participant