-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[JSC] Extend JSBigInt::compare and JSBigInt::compareToDouble for Int64 and Int32 #25964
[JSC] Extend JSBigInt::compare and JSBigInt::compareToDouble for Int64 and Int32 #25964
Conversation
EWS run on previous version of this PR (hash d3c1a9f) |
d3c1a9f
to
7b2e4f7
Compare
EWS run on previous version of this PR (hash 7b2e4f7) |
7b2e4f7
to
b2bdf54
Compare
b2bdf54
to
b94ac0b
Compare
EWS run on previous version of this PR (hash b94ac0b)
|
b94ac0b
to
db1d04d
Compare
EWS run on previous version of this PR (hash db1d04d)
|
db1d04d
to
85d75f2
Compare
EWS run on previous version of this PR (hash 85d75f2)
|
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.
r=me with recommendations.
85d75f2
to
71c655c
Compare
EWS run on previous version of this PR (hash 71c655c)
|
71c655c
to
b5ea7cf
Compare
EWS run on previous version of this PR (hash b5ea7cf)
|
b5ea7cf
to
0d724d8
Compare
EWS run on previous version of this PR (hash 0d724d8)
|
0d724d8
to
10b9352
Compare
EWS run on current version of this PR (hash 10b9352)
|
10b9352
to
8ad48fd
Compare
β¦4 and Int32 https://bugs.webkit.org/show_bug.cgi?id=271073 rdar://124706111 Reviewed by Keith Miller. This patch extends JSBigInt::compare and JSBigInt::compareToDouble for Int64 and Int32. * Source/JavaScriptCore/runtime/JSBigInt.cpp: (JSC::Int64BigIntImpl::Int64BigIntImpl): (JSC::Int64BigIntImpl::isZero): (JSC::Int64BigIntImpl::sign): (JSC::Int64BigIntImpl::length): (JSC::Int64BigIntImpl::digit): (JSC::JSBigInt::compare): (JSC::JSBigInt::compareToDouble): * Source/JavaScriptCore/runtime/JSBigInt.h: Canonical link: https://commits.webkit.org/276417@main
8ad48fd
to
f706066
Compare
Committed 276417@main (f706066): https://commits.webkit.org/276417@main Reviewed commits have been landed. Closing PR #25964 and removing active labels. |
f706066
8ad48fd