Skip to content

Conversation

@koteelok
Copy link

In some places, you use new Array(size) to make number arrays. But JS fills the array with empty space and V8 will treat them as HOLEY_SMI_ELEMENTS and HOLEY_DOUBLE_ELEMENTS instead of PACKED_SMI_ELEMENTS and PACKED_DOUBLE_ELEMENTS.

I grepped through codebase and fixed this in some places, but I don’t think it will result in any noticeable speed-up. I just "scratched the itch"

Feel free to reject, if you don't care :P

@davidetan davidetan self-assigned this Oct 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants