Skip to content

Commit

Permalink
deps: V8: cherry-pick a3cc8522a4c8
Browse files Browse the repository at this point in the history
Original commit message:

    [riscv] avoid cpu probing in li_ptr

    CPU probing is an expensive thing to do and we should avoid doing it upon every li_ptr call.

    Fixes performance regresion bisected in riscv-forks/electron#1

    Change-Id: Ib5ff89b2a730e08de6735123ae60adeffe811ed8
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/5612950
    Commit-Queue: Yahan Lu <[email protected]>
    Reviewed-by: Yahan Lu <[email protected]>
    Reviewed-by: Michael Lippautz <[email protected]>
    Cr-Commit-Position: refs/heads/main@{#94349}

Refs: v8/v8@a3cc852
PR-URL: #53412
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
  • Loading branch information
kxxt authored and targos committed Jun 20, 2024
1 parent 2defaaf commit bd68888
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion common.gypi
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@

# Reset this number to 0 on major V8 upgrades.
# Increment by one for each non-official patch applied to deps/v8.
'v8_embedder_string': '-node.14',
'v8_embedder_string': '-node.15',

##### V8 defaults for Node.js #####

Expand Down
1 change: 1 addition & 0 deletions deps/v8/AUTHORS
Original file line number Diff line number Diff line change
Expand Up @@ -181,6 +181,7 @@ Kevin Gibbons <[email protected]>
Keyhan Vakil <[email protected]>
Kris Selden <[email protected]>
Kyounga Ra <[email protected]>
Levi Zim <[email protected]>
Loo Rong Jie <[email protected]>
Lu Yahan <[email protected]>
Ludovic Mermod <[email protected]>
Expand Down
1 change: 0 additions & 1 deletion deps/v8/src/codegen/riscv/assembler-riscv.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1087,7 +1087,6 @@ void Assembler::GeneralLi(Register rd, int64_t imm) {
}

void Assembler::li_ptr(Register rd, int64_t imm) {
base::CPU cpu;
// Initialize rd with an address
// Pointers are 48 bits
// 6 fixed instructions are generated
Expand Down

0 comments on commit bd68888

Please sign in to comment.