-
Notifications
You must be signed in to change notification settings - Fork 203
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
Support wasm64 #444
Open
trcrsired
wants to merge
57
commits into
WebAssembly:main
Choose a base branch
from
trcrsired:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Support wasm64 #444
Changes from 13 commits
Commits
Show all changes
57 commits
Select commit
Hold shift + click to select a range
37122df
make api.h to support wasm64
trcrsired 8d11df7
wasilibc_real support wasm64
trcrsired 37702e7
fix api.h
trcrsired 59a8228
fix atomic_arch for wasm64
trcrsired 10dd8a7
initial attempt to fix c_header.rs
trcrsired d146432
Add support for 64 bits
trcrsired 2659f2a
fix __wasi_thread_spawn's return type
trcrsired 3b15bd1
change header of api.h to temporarily manually maintained
trcrsired 48e8b25
Fix formatting issue for rustfmt
trcrsired a5ad684
Remove all signal apis
trcrsired b9d2738
Fix symbol of thread_spawn
trcrsired 72d9602
Add wasi64 and wasi64+thread to CI
trcrsired dfc09f2
Change comment for CI for wasi64 and thread
trcrsired c9ad745
Merge wasm32 code with wasm64 code
trcrsired 8a294b8
change comment in main.yml from wasi64 to wasm64
trcrsired 46ed3a8
Fix c_header.rs issue with rust fmt
trcrsired 7bd36bb
Merge branch 'WebAssembly:main' into main
trcrsired 4877ea7
support dlfcn.h for wasm64
trcrsired 56fdd62
Merge branch 'WebAssembly:main' into main
trcrsired a814fa8
add import name for wasi64 with _i64 postfix
trcrsired d6aac51
generate __wasilibc_real.c from rust code
trcrsired 9703cc1
should be _i64 not -i64
trcrsired d2884b2
fix __wasilibc_real.c
trcrsired 26f0de5
__WASI_NOEXCEPT missing in the definition of function
trcrsired 3f36fea
fix __WASI_NOEXCEPT missing issue
trcrsired c8352f8
fix the issue that the definition of one_polloff isn't right on wasm64
trcrsired 4f01294
fix the issue with rustfmt
trcrsired c889960
change _i64 postfix to _wasm64
trcrsired 3289596
Merge branch 'WebAssembly:main' into main
trcrsired b08a02c
Merge remote-tracking branch 'parent/main'
trcrsired e6abb3c
add wasm64-wasi-preview2
trcrsired 5c5b14e
Add wasm64-wasi-preview2 expected
trcrsired c94c0c6
fix CI and wasm32-wasi-preview2 predefined-macros
trcrsired 01e6ed3
prevent clang 10 from crashing
trcrsired 79184bf
Merge remote-tracking branch 'parent/main'
trcrsired 84472d9
recompile to fix CI
trcrsired b40e999
Merge branch 'WebAssembly:main' into main
trcrsired 12fc7a2
Merge remote-tracking branch 'parent/main'
trcrsired 96b3d9e
fix
trcrsired 34e1985
fix stdint.h on 64 bit wasm
trcrsired ce67b68
Fix makefile for new the p2 update
trcrsired 3c387fa
Remove old preview triple
trcrsired 2672d35
fix wasi32-wasi
trcrsired e027353
Merge branch 'WebAssembly:main' into main
trcrsired cabd676
Merge branch 'WebAssembly:main' into main
trcrsired 3fa33a2
Merge branch 'WebAssembly:main' into main
trcrsired 45b8471
Merge remote-tracking branch 'parent/main'
trcrsired bc5755d
remove -D_WASI_EMULATED_PTHREAD since it breaks llvm even we enable p…
trcrsired 66491d2
give back shit
trcrsired 0705ecf
force enable feature
trcrsired b129381
this is dogshit
trcrsired 3992e73
fix
trcrsired 0873325
fix
trcrsired 8f8dfe7
fix issues
trcrsired 6966fb5
Merge remote-tracking branch 'parent/main'
trcrsired bb8e70a
fix .s not .S
trcrsired e813115
Fix thread build
trcrsired File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe
BUILD_WASM64
would better match the target name?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what about just WASI64?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But WASI64 is not a thing as far as I know. It doesn't mean much to me as the reader.. whereas WASM64 is the name of the llvm target.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
but wasm64 is not wasi64, because you also have wasm64-emscripten.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what about just allowing users to set TRIPLE?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed to WASM64