Skip to content

[v25.x] deps: V8: cherry-pick b25cd62c7ba2#62354

Open
mertcanaltin wants to merge 1 commit intonodejs:v25.x-stagingfrom
mertcanaltin:mert/v8-backport-utf8length-simdutf-v2
Open

[v25.x] deps: V8: cherry-pick b25cd62c7ba2#62354
mertcanaltin wants to merge 1 commit intonodejs:v25.x-stagingfrom
mertcanaltin:mert/v8-backport-utf8length-simdutf-v2

Conversation

@mertcanaltin
Copy link
Member

@mertcanaltin mertcanaltin commented Mar 20, 2026 •
edited
Loading

Backport V8 upstream commit b25cd62c7ba2 using git node v8 backport.

refs:
#61601
#62349

original commit: v8/v8@b25cd62c7ba2 @anonrig

fyi @aduh95 @Renegade334

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Mar 20, 2026

Review requested:

  • @nodejs/gyp
  • @nodejs/security-wg
  • @nodejs/v8-update

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. v8 engine Issues and PRs related to the V8 dependency. labels Mar 20, 2026
@mertcanaltin mertcanaltin mentioned this pull request Mar 20, 2026
Closed
@aduh95 aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels Mar 20, 2026
@anonrig
Copy link
Member

anonrig commented Mar 20, 2026

For reference here's the original commit: v8/v8@b25cd62c7ba2

@Renegade334
Copy link
Member

Renegade334 commented Mar 20, 2026 •
edited
Loading

Probably better if the base were changed to v25.x-staging rather than main, so as not to clash with the V8 update PR for v26.x, otherwise LGTM.

@mertcanaltin mertcanaltin changed the base branch from main to v25.x-staging March 20, 2026 16:03
@mertcanaltin mertcanaltin requested a review from a team as a code owner March 20, 2026 16:03
@mertcanaltin
Copy link
Member Author

mertcanaltin commented Mar 20, 2026 •
edited
Loading

Probably better if the base were changed to v25.x-staging rather than main, so as not to clash with the V8 update PR for v26.x, otherwise LGTM.

Good point, thanks, I will edit rebase problem

@Renegade334
Copy link
Member

Renegade334 commented Mar 20, 2026 •
edited
Loading

(you'll need to reset your local branch to v25.x-staging, cherry-pick your backport commit 22a6e58 to it, and force push)

@anonrig @mertcanaltin
deps: V8: cherry-pick b25cd62c7ba2
c13a613 
Original commit message:

    Optimize utf8length for utf16 values

    Change-Id: I4d9546ab45264852f52361123e86fa3b38d79e45
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/7142019
    Reviewed-by: Anton Bikineev <bikineev@chromium.org>
    Reviewed-by: Erik Corry <erikcorry@chromium.org>
    Commit-Queue: Erik Corry <erikcorry@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#103754}

Refs: v8/v8@b25cd62
@mertcanaltin mertcanaltin force-pushed the mert/v8-backport-utf8length-simdutf-v2 branch from 22a6e58 to c13a613 Compare March 20, 2026 16:12
@mertcanaltin
Copy link
Member Author

mertcanaltin commented Mar 20, 2026

(you'll need to reset your local branch to v25.x-staging, cherry-pick your backport commit 22a6e58 to it, and force push)

solved 🎉

@Renegade334 Renegade334 added the v25.x Issues that can be reproduced on v25.x or PRs targeting the v25.x-staging branch. label Mar 20, 2026
@Renegade334 Renegade334 changed the title deps: V8: cherry-pick b25cd62c7ba2 [v25.x] deps: V8: cherry-pick b25cd62c7ba2 Mar 20, 2026
@aduh95
Copy link
Contributor

aduh95 commented Mar 20, 2026

Probably better if the base were changed to v25.x-staging rather than main, so as not to clash with the V8 update PR for v26.x, otherwise LGTM.

I disagree, it's much better to land on main unless it cannot be done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@anonrig anonrig anonrig approved these changes

@Renegade334 Renegade334 Renegade334 approved these changes

@aduh95 aduh95 aduh95 approved these changes

Assignees

No one assigned

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. needs-ci PRs that need a full CI run. request-ci Add this label to start a Jenkins CI on a PR. v8 engine Issues and PRs related to the V8 dependency. v25.x Issues that can be reproduced on v25.x or PRs targeting the v25.x-staging branch.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@mertcanaltin @nodejs-github-bot @anonrig @Renegade334 @aduh95