Skip to content

Commit 09e4182

Browse files
Milad Farazmandtargos
Milad Farazmand
authored andcommitted
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 <[email protected]> Commit-Queue: Milad Farazmand <[email protected]> Cr-Commit-Position: refs/heads/master@{#65644} Refs: v8/v8@d89f4ef PR-URL: #31354 Reviewed-By: David Carlier <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Beth Griggs <[email protected]> Reviewed-By: Richard Lau <[email protected]>
1 parent 8252c06 commit 09e4182

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

common.gypi

+1-1
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@
3838

3939
# Reset this number to 0 on major V8 upgrades.
4040
# Increment by one for each non-official patch applied to deps/v8.
41-
'v8_embedder_string': '-node.29',
41+
'v8_embedder_string': '-node.30',
4242

4343
##### V8 defaults for Node.js #####
4444

deps/v8/src/codegen/s390/macro-assembler-s390.cc

+2-2
Original file line numberDiff line numberDiff line change
@@ -1336,8 +1336,8 @@ void MacroAssembler::CheckDebugHook(Register fun, Register new_target,
13361336
ExternalReference debug_hook_active =
13371337
ExternalReference::debug_hook_on_function_call_address(isolate());
13381338
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);
13411341

13421342
{
13431343
// Load receiver to pass it later to DebugOnFunctionCall hook.

0 commit comments

Comments
 (0)