Skip to content

Wasm modules initial #1274

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 2 commits into from
Nov 5, 2022
Merged

Wasm modules initial #1274

merged 2 commits into from
Nov 5, 2022

Conversation

ubaidsk
Copy link
Collaborator

@ubaidsk ubaidsk commented Nov 5, 2022

Corresponding LFortran PR - lfortran/lfortran#992.

@ubaidsk ubaidsk requested a review from certik November 5, 2022 16:47
@ubaidsk
Copy link
Collaborator Author

ubaidsk commented Nov 5, 2022

Both the modules_01.py and modules_02.py in integration_tests currently work with the wasm backend.

(lp) ubaid@ubaid-Lenovo-ideapad-330-15ARR:~/OpenSource/lpython$ lpython integration_tests/modules_01.py --backend llvm
OK
(lp) ubaid@ubaid-Lenovo-ideapad-330-15ARR:~/OpenSource/lpython$ lpython integration_tests/modules_01.py --backend wasm && node a.out.js
OK
(lp) ubaid@ubaid-Lenovo-ideapad-330-15ARR:~/OpenSource/lpython$ lpython integration_tests/modules_02.py --backend llvm
g()
f()
(lp) ubaid@ubaid-Lenovo-ideapad-330-15ARR:~/OpenSource/lpython$ lpython integration_tests/modules_02.py --backend wasm && node a.out.js
g()
f()
(lp) ubaid@ubaid-Lenovo-ideapad-330-15ARR:~/OpenSource/lpython$ 

@ubaidsk
Copy link
Collaborator Author

ubaidsk commented Nov 5, 2022

This is ready. Please possibly review and please share feedback.

Copy link
Contributor

@certik certik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this looks good, thanks!

@ubaidsk ubaidsk merged commit d4a6d53 into lcompilers:main Nov 5, 2022
@ubaidsk ubaidsk deleted the wasm_modules_initial branch November 5, 2022 20:55
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