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.

Sign up for GitHub

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

Jump to bottom

hash(::BigInt) broken in 1.9 on 32-bit systems #50075

Closed
LilithHafner opened this issue Jun 5, 2023 · 2 comments · Fixed by #50076
Closed

hash(::BigInt) broken in 1.9 on 32-bit systems #50075

LilithHafner opened this issue Jun 5, 2023 · 2 comments · Fixed by #50076
Labels
bug Indicates an unexpected problem or unintended behavior hashing

Comments

@LilithHafner
Copy link
Member

LilithHafner commented Jun 5, 2023 •
edited
Loading 

julia> a = 12345678901234
12345678901234

julia> b = big(a)
12345678901234

julia> a == b
true

julia> hash(a) == hash(b)
false

julia> versioninfo()
Julia Version 1.9.0
Commit 8e63055292 (2023-05-07 11:25 UTC)
Platform Info:
  OS: Windows (i686-w64-mingw32)
  CPU: 20 × 12th Gen Intel(R) Core(TM) i7-12700H
  WORD_SIZE: 32
  LIBM: libopenlibm
  LLVM: libLLVM-14.0.6 (ORCJIT, alderlake)
  Threads: 1 on 20 virtual cores

Presumably also broken on master.
@LilithHafner LilithHafner added bug Indicates an unexpected problem or unintended behavior hashing labels Jun 5, 2023
@LilithHafner LilithHafner mentioned this issue Jun 5, 2023
Merged
@LilithHafner LilithHafner changed the title hash(::BigInt) broken in 1.9 hash(::BigInt) broken in 1.9 on 32-bit systems Jun 5, 2023
@oscardssmith
Copy link
Member

Specifically the problem with it is that it is loading UInt rather UInt64. Correct?

@oscardssmith oscardssmith mentioned this issue Jun 5, 2023
Merged
@oscardssmith
Copy link
Member

hopefully #50076 fixes it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Indicates an unexpected problem or unintended behavior hashing
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix hash(::BigInt) on 32 bit systems oscardssmith/julia
2 participants
@oscardssmith @LilithHafner