Skip to content
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

avoid #8231, bitwise move to mul,div #15070

Merged
merged 4 commits into from
Aug 25, 2020
Merged

avoid #8231, bitwise move to mul,div #15070

merged 4 commits into from
Aug 25, 2020

Conversation

bung87
Copy link
Collaborator

@bung87 bung87 commented Jul 25, 2020

avoiding #8231
"bitwise operators in JS are 32-bit" as @Yardanico said
the PNode .intVal store as BigestInt which currently is int64 which I dont want touch it, instead changing bitwise move operation to mul and div.

@narimiran
Copy link
Member

Can you write a test for #8231?

compiler/jsgen.nim Outdated Show resolved Hide resolved
@Araq Araq merged commit 7cee63b into nim-lang:devel Aug 25, 2020
mildred pushed a commit to mildred/Nim that referenced this pull request Jan 11, 2021
* avoid nim-lang#8231, bitwise move to mul,div

* add test for nim-lang#8231

* fix bitwise move when div result is float

* bitwise move depends on typ.size
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.

3 participants