-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Don't use le32/le64 #8344
Merged
Merged
Don't use le32/le64 #8344
Changes from 30 commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
ddd0189
Use wasm32/wasm64 instead of le32/le64
steven-johnson 60b46ee
Update Makefile
steven-johnson 3cce91f
Update CMakeLists.txt
steven-johnson 24fd1b2
Update CMakeLists.txt
steven-johnson 67d80ae
trigger buildbots
steven-johnson 301c610
Try x86 instead of wasm
steven-johnson 09fba5f
-Wno-sync-alignment
steven-johnson af4af91
Revert "-Wno-sync-alignment"
abadams 8002729
Don't do 64-bit atomics in 32-bit runtimes.
abadams 30a755c
Reapply "-Wno-sync-alignment"
steven-johnson 2cfd588
Revert "Don't do 64-bit atomics in 32-bit runtimes."
steven-johnson 9325a2b
try arm instead
steven-johnson a9b0b61
sync warning
steven-johnson 4e5bd58
gnueabihf
steven-johnson bfbe718
gnueabihf
steven-johnson 2e8a6e8
Comments
steven-johnson 6401755
Update CMakeLists.txt
steven-johnson def7da0
Let's try x86 instead
steven-johnson 0e766ae
-Wno-sync-alignment
steven-johnson 3e99fb8
Back to wasm
steven-johnson 678164f
Update LLVM_Runtime_Linker.cpp
steven-johnson 0c050b2
Fix RUNTIME_TRIPLE_WIN_GENERIC_64
steven-johnson 3ca975b
Reenable warning
steven-johnson 622c9cd
Update LLVM_Runtime_Linker.cpp
steven-johnson 9885b30
Update LLVM_Runtime_Linker.cpp
steven-johnson 6b6a76e
Merge branch 'main' into srj/no-le64
steven-johnson 5b0ccc0
Update LLVM_Runtime_Linker.cpp
steven-johnson c88ab38
Merge branch 'main' into srj/no-le64
steven-johnson fe54bfe
Fixes
steven-johnson b28acbc
warnings
steven-johnson 97046ce
wasm64->x86_64
steven-johnson 007186b
back to wasm64
steven-johnson 38f9ab6
Use wasm32/wasm64 for webgpu
steven-johnson 2d5033b
Update Makefile
steven-johnson 39cd51d
trigger buildbots
steven-johnson 55c3610
Revert "Update Makefile"
steven-johnson a2064e4
Revert "Use wasm32/wasm64 for webgpu"
steven-johnson 09c86a2
Revert "back to wasm64"
steven-johnson abbc1cd
-fno-jump-tables
steven-johnson b9497bd
Reapply "Use wasm32/wasm64 for webgpu"
steven-johnson d6affab
tabs
steven-johnson 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
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.
Is this change still needed?
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.
No, removing
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.
Update: yes, it is needed now