Yes, that looks like a bug. I'll make a fix. Thanks for the report!

On Fri, Feb 28, 2025 at 4:19 AM ClearScript Developers <
[email protected]> wrote:

> Greetings!
>
> Consider the line here
> <https://github.com/v8/v8/blob/f5fba01dabb7f8677c3e2e737c9017fe6576f041/src/objects/bigint.cc#L1376>
> :
>
>     if (kDigitBits == 32 && words[words64_count - 1] <= (1ULL << 32))
> length--;
>
> It seems like the goal here is to discard the final 32-bit "half-word" if
> it's zero. If that's correct, shouldn't the comparison operator be "<"
> rather than "<="?
>
> Thanks!
>

-- 
-- 
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- 
You received this message because you are subscribed to the Google Groups 
"v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion visit 
https://groups.google.com/d/msgid/v8-dev/CAKSzg3SF06d1PHsDavw_G5irPc9et%2BoHau3DkxggYiZZqvfyRg%40mail.gmail.com.

Reply via email to