-
Notifications
You must be signed in to change notification settings - Fork 15.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
feat: enable pointer compression and 31bit smis #21468
Conversation
Can we align the patch based on nodejs/node@dda658c , so that we only have to enable the gyp variables and the definitions part need not be maintained. |
95db328
to
8e62c13
Compare
@deepak1556 done, sort of. I'm not sure if I got the gyp syntax correct, but presumably CI will tell me :D |
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.
LGTM
CI is flake |
Release Notes Persisted
|
Closes #21464
npm test
passesNotes: Enabled V8 pointer compression.