Skip to content

Merge SwiftWasm build scripts, thin CMO to 5.4 release #2765

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

Merged
merged 32 commits into from
Mar 15, 2021

Conversation

yonihemi
Copy link
Member

No description provided.

@yonihemi yonihemi marked this pull request as draft February 18, 2021 12:13
@kateinoigakukun
Copy link
Member

@yonihemi This compiler crash of Concurrency module can be resolved by merging the latest upstream 5.4 branch, I think. I faced the same crash on the main stream.

@yonihemi
Copy link
Member Author

You mean based on PR #2756 and with concurrency enabled, right?
This would require your ff78f61 first, I guess.

@kateinoigakukun
Copy link
Member

kateinoigakukun commented Feb 19, 2021

You mean based on PR #2756 and with concurrency enabled, right?

Right

@kateinoigakukun
Copy link
Member

kateinoigakukun commented Mar 7, 2021

@yonihemi Sorry for my slow response 🙇 I think cherry-picking swiftlang#35220 this patch will solve current build failure.

yonihemi added 3 commits March 8, 2021 11:00
…@kateinoigakukun]

If function pointer is a direct pointer to a function,
it could not be an async function pointer. And additive
operation to function address is illegal on some archs
like wasm32, so emit undefs instead.
@kateinoigakukun
Copy link
Member

You need to pick this also 2b7d31b

@yonihemi yonihemi marked this pull request as ready for review March 15, 2021 03:51
@yonihemi
Copy link
Member Author

Thank you @kateinoigakukun!

@yonihemi yonihemi merged commit e3c2913 into swiftwasm-release/5.4 Mar 15, 2021
@yonihemi yonihemi deleted the swiftwasm-release/5.4-tooling branch March 15, 2021 07:09
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.

3 participants