Skip to content

Fix sil-func-extractor and driver #28

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 4 commits into from
Dec 25, 2019

Conversation

kateinoigakukun
Copy link
Member

@kateinoigakukun kateinoigakukun commented Dec 24, 2019

  • Use llvm-ar instead of ar because ar doesn't support latest webassembly archiving
  • Disable objc-interop of sil-func-extractor to pass some test cases
    • clang doesn't support Objective-C using wasm obejct file format
    • Crashed here
    • And there is no plan to support it
  • Exclude test cases which use -enable-objc-interop flag due to same reason.

@kateinoigakukun kateinoigakukun marked this pull request as ready for review December 24, 2019 16:31
@kateinoigakukun kateinoigakukun merged commit c0c3241 into swiftwasm:swiftwasm Dec 25, 2019
kateinoigakukun added a commit that referenced this pull request Jan 11, 2020
* [WASM] Remove rpath flag and link objects using llvm-ar instead of ar

* [WASM] Disable objc interop for sil-func-extractor when targeting wasm

* [WASM] Exclude test cases which use -enable-objc-interop

* [WASM] Make availability_returns_twice.swift unsupoorted on wasi due to lack of setjmp
@kateinoigakukun kateinoigakukun deleted the katei/fix-driver branch January 28, 2020 12:46
MaxDesiatov pushed a commit that referenced this pull request Jan 29, 2020
* [WASM] Remove rpath flag and link objects using llvm-ar instead of ar

* [WASM] Disable objc interop for sil-func-extractor when targeting wasm

* [WASM] Exclude test cases which use -enable-objc-interop

* [WASM] Make availability_returns_twice.swift unsupoorted on wasi due to lack of setjmp
MaxDesiatov pushed a commit that referenced this pull request Jan 29, 2020
[pull] swiftwasm from apple:master
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