FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

WASM, WASM_X64: Support shift operations by ubaidsk · Pull Request #1482 · lcompilers/lpython · GitHub

/ lpython Public

WASM, WASM_X64: Support shift operations - #1482

Merged
ubaidsk merged 4 commits into
lcompilers:mainfrom
ubaidsk:wasm_shift_opts
Feb 1, 2023
ConversationCommits (4)ChecksFiles changed
Merged

WASM, WASM_X64: Support shift operations#1482
ubaidsk merged 4 commits into
lcompilers:mainfrom
ubaidsk:wasm_shift_opts

Conversation

ubaidsk commented Jan 31, 2023 •
edited
Loading

Copy link
Copy Markdown
Collaborator

This PR adds support of left shift and right shift operations in the wasm and wasm_x64 backends. For right shift, I currently used signed/arithmetic right shift operation (this seems similar to the llvm backend). Other possibility is using unsigned/logical right shift operation.

ubaidsk added the wasm label Jan 31, 2023
ubaidsk requested a review from certik January 31, 2023 16:54

ubaidsk commented Jan 31, 2023

Copy link
Copy Markdown
Collaborator Author

This is ready. Please possibly review and share feedback.

ubaidsk commented Jan 31, 2023

Copy link
Copy Markdown
Collaborator Author

This PR might fail after merging of #1481. As the test integration_tests/bit_operations_i32.py was enabled for the wasm_x64 backend in the commit TEST: Enable suporting tests and the wasm_x64 backend currently does not have support for shift operations.

certik mentioned this pull request Jan 31, 2023
Open
23 tasks

certik commented Jan 31, 2023

Copy link
Copy Markdown
Contributor

All tests passed and I now rebased on top of the latest master.

ubaidsk changed the title WASM: Support shift operations WASM, WASM_X64: Support shift operations Feb 1, 2023

ubaidsk commented Feb 1, 2023

Copy link
Copy Markdown
Collaborator Author

The tests should hopefully pass now.

The new commits X86Assembler: Implement SHL and SAR Instructions and
WASM_X64: Support I32Shl() and I32ShrS() need review.

ubaidsk commented Feb 1, 2023

Copy link
Copy Markdown
Collaborator Author

I am adding this to auto-merge as it seems approved. I will support changes/updates suggested (if any) for recently added commits in a new PR.

ubaidsk enabled auto-merge February 1, 2023 06:47
ubaidsk merged commit 9417d7e into lcompilers:main Feb 1, 2023
ubaidsk deleted the wasm_shift_opts branch February 1, 2023 07:44

certik commented Feb 1, 2023

Copy link
Copy Markdown
Contributor

Yes, this is good, thanks!

This file contains hidden or 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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

certik certik approved these changes

Assignees

No one assigned

Labels

wasm

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL