Skip to content

Temporarily mark swiftasync unsupported for x86_64 Windows #2692

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

varungandhi-apple
Copy link

The new ABI needs stack adjustment, but on Windows, swifttailcc only
works if there is no stack adjustment.

@varungandhi-apple
Copy link
Author

@swift-ci test

The new ABI needs stack adjustment, but on Windows, swifttailcc only
works if there is no stack adjustment.
@varungandhi-apple
Copy link
Author

@swift-ci test

@varungandhi-apple varungandhi-apple merged commit 73e4a09 into swiftlang:apple/stable/20210107 Mar 16, 2021
@varungandhi-apple varungandhi-apple deleted the vg-apple/stable branch March 16, 2021 20:01
aschwaighofer pushed a commit to aschwaighofer/llvm-project that referenced this pull request Mar 17, 2021
Temporarily mark swiftasync unsupported for x86_64 Windows

(cherry picked from commit 73e4a09)
@aschwaighofer
Copy link

CC @compnerd

@aschwaighofer
Copy link

https://github.com/apple/llvm-project/blob/swift/main/llvm/lib/Target/X86/X86FrameLowering.cpp#L1333

Is the problematic line.

After swiftlang/swift#36281 we emit variable number of llvm arguments between async caller and callee so we will trigger that assert.

shahmishal added a commit that referenced this pull request Mar 17, 2021
Cherry-pick to swift/main: Merge pull request #2692 from varungandhi-apple/vg-apple/stable
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