Commit b724eaf
deps: V8: cherry-pick d89f4ef1cd62
Original commit message:
S390x: improve performance by skipping Debug Hook if not needed
Change-Id: Ib4b2821f2941cdc131f9c75b89a3baced7554f8d
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1991802
Reviewed-by: Junliang Yan <jyan@ca.ibm.com>
Commit-Queue: Milad Farazmand <miladfar@ca.ibm.com>
Cr-Commit-Position: refs/heads/master@{#65644}
Refs: v8/v8@d89f4ef
PR-URL: #31354
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>1 parent 655d068 commit b724eaf
2 files changed
Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -38,7 +38,7 @@ | |||
| 38 | 38 | ||
| 39 | 39 | # Reset this number to 0 on major V8 upgrades. | |
| 40 | 40 | # Increment by one for each non-official patch applied to deps/v8. | |
| 41 | - 'v8_embedder_string': '-node.29', | ||
| 41 | + 'v8_embedder_string': '-node.30', | ||
| 42 | 42 | ||
| 43 | 43 | ##### V8 defaults for Node.js ##### | |
| 44 | 44 | ||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1336,8 +1336,8 @@ void MacroAssembler::CheckDebugHook(Register fun, Register new_target, | |||
| 1336 | 1336 | ExternalReference debug_hook_active = | |
| 1337 | 1337 | ExternalReference::debug_hook_on_function_call_address(isolate()); | |
| 1338 | 1338 | Move(r6, debug_hook_active); | |
| 1339 | - tm(MemOperand(r6), Operand::Zero()); | ||
| 1340 | - bne(&skip_hook); | ||
| 1339 | + tm(MemOperand(r6), Operand(0xFF)); | ||
| 1340 | + beq(&skip_hook); | ||
| 1341 | 1341 | ||
| 1342 | 1342 | { | |
| 1343 | 1343 | // Load receiver to pass it later to DebugOnFunctionCall hook. | |
0 commit comments