-
-
Notifications
You must be signed in to change notification settings - Fork 33.9k
deps: V8: cherry-pick highway@989a498fdf3 #60682
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
Merged
nodejs-github-bot
merged 1 commit into
nodejs:main
from
richardlau:backport-highway-fix-gcc15
Nov 13, 2025
Merged
deps: V8: cherry-pick highway@989a498fdf3 #60682
nodejs-github-bot
merged 1 commit into
nodejs:main
from
richardlau:backport-highway-fix-gcc15
Nov 13, 2025
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
|
Review requested:
|
targos
approved these changes
Nov 12, 2025
Collaborator
Collaborator
Collaborator
Collaborator
cjihrig
approved these changes
Nov 12, 2025
lpinca
approved these changes
Nov 12, 2025
meixg
approved these changes
Nov 13, 2025
Member
Author
Member
|
@richardlau thanks, I just landed the V8 update. |
Original commit message:
GCC 15 removed avx10.2-512 target
PiperOrigin-RevId: 823560321
Refs: google/highway@989a498
a10eaa3 to
5c19834
Compare
Member
Author
|
Rebased and renumbered the embedder string. |
targos
approved these changes
Nov 13, 2025
Collaborator
Collaborator
Collaborator
Collaborator
Collaborator
|
Landed in da71ab6 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
author ready
PRs that have at least one approval, no pending requests for changes, and a CI started.
build
Issues and PRs related to build files or the CI.
dont-land-on-v20.x
PRs that should not land on the v20.x-staging branch and should not be released in v20.x.
dont-land-on-v22.x
PRs that should not land on the v22.x-staging branch and should not be released in v22.x.
dont-land-on-v24.x
PRs that should not land on the v24.x-staging branch and should not be released in v24.x.
dont-land-on-v25.x
PRs that should not land on the v25.x-staging branch and should not be released in v25.x.
needs-ci
PRs that need a full CI run.
v8 engine
Issues and PRs related to the V8 dependency.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Original commit message:
Refs: google/highway@989a498
Fixes: #60566
cc @nodejs/v8-update
This hasn't landed in upstream V8 yet, so the usual
@node-core/utilsV8 backport tooling can't be used. I've tried to replicate the commit message format. Hopefully V8 will roll the highway dependency forward to pick up the fix so we won't have to float the patch forever.