FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

deps: patch V8 to 9.0.257.16 · nodejs/node@5b358d5 · GitHub

/ node Public

Commit 5b358d5

Browse files
targos
committed
deps: patch V8 to 9.0.257.16
Refs: v8/v8@9.0.257.13...9.0.257.16 PR-URL: #38218 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: Beth Griggs <bgriggs@redhat.com>
1 parent f1a21e5 commit 5b358d5

6 files changed

Lines changed: 47 additions & 17 deletions

File tree

‎deps/v8/include/v8-version.h‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
#define V8_MAJOR_VERSION 9
1212
#define V8_MINOR_VERSION 0
1313
#define V8_BUILD_NUMBER 257
14-
#define V8_PATCH_LEVEL 13
14+
#define V8_PATCH_LEVEL 16
1515

1616
// Use 1 for candidates and 0 otherwise.
1717
// (Boolean macro values are not supported by all preprocessors.)

‎deps/v8/src/compiler/js-call-reducer.cc‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4907,6 +4907,14 @@ Reduction JSCallReducer::ReduceStringPrototypeIndexOf(Node* node) {
49074907
Node* position = n.Argument(1);
49084908
new_position = effect = graph()->NewNode(
49094909
simplified()->CheckSmi(p.feedback()), position, effect, control);
4910+
4911+
Node* receiver_length =
4912+
graph()->NewNode(simplified()->StringLength(), new_receiver);
4913+
new_position = graph()->NewNode(
4914+
simplified()->NumberMin(),
4915+
graph()->NewNode(simplified()->NumberMax(), new_position,
4916+
jsgraph()->ZeroConstant()),
4917+
receiver_length);
49104918
}
49114919

49124920
NodeProperties::ReplaceEffectInput(node, effect);

‎deps/v8/src/debug/debug.cc‎

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1641,8 +1641,15 @@ Handle<Object> Debug::FindSharedFunctionInfoInScript(Handle<Script> script,
16411641
UnoptimizedCompileFlags::ForScriptCompile(isolate_, *script);
16421642
ParseInfo parse_info(isolate_, flags, &compile_state);
16431643
IsCompiledScope is_compiled_scope;
1644-
Compiler::CompileToplevel(&parse_info, script, isolate_,
1645-
&is_compiled_scope);
1644+
const MaybeHandle<SharedFunctionInfo> maybe_result =
1645+
Compiler::CompileToplevel(&parse_info, script, isolate_,
1646+
&is_compiled_scope);
1647+
if (maybe_result.is_null()) {
1648+
if (isolate_->has_pending_exception()) {
1649+
isolate_->clear_pending_exception();
1650+
}
1651+
break;
1652+
}
16461653
continue;
16471654
}
16481655
// We found it if it's already compiled.

‎deps/v8/src/wasm/baseline/ia32/liftoff-assembler-ia32.h‎

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4027,13 +4027,8 @@ void LiftoffAssembler::emit_i64x2_mul(LiftoffRegister dst, LiftoffRegister lhs,
40274027
Pmuludq(tmp2.fp(), tmp2.fp(), lhs.fp());
40284028
Paddq(tmp2.fp(), tmp2.fp(), tmp1.fp());
40294029
Psllq(tmp2.fp(), tmp2.fp(), 32);
4030-
if (CpuFeatures::IsSupported(AVX)) {
4031-
CpuFeatureScope scope(this, AVX);
4032-
vpmuludq(dst.fp(), lhs.fp(), rhs.fp());
4033-
} else {
4034-
if (dst.fp() != lhs.fp()) movaps(dst.fp(), lhs.fp());
4035-
pmuludq(dst.fp(), rhs.fp());
4036-
}
4030+
liftoff::EmitSimdCommutativeBinOp<&Assembler::vpmuludq, &Assembler::pmuludq>(
4031+
this, dst, lhs, rhs);
40374032
Paddq(dst.fp(), dst.fp(), tmp2.fp());
40384033
}
40394034

‎deps/v8/src/wasm/baseline/x64/liftoff-assembler-x64.h‎

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3559,13 +3559,8 @@ void LiftoffAssembler::emit_i64x2_mul(LiftoffRegister dst, LiftoffRegister lhs,
35593559
Pmuludq(tmp2.fp(), lhs.fp());
35603560
Paddq(tmp2.fp(), tmp1.fp());
35613561
Psllq(tmp2.fp(), 32);
3562-
if (CpuFeatures::IsSupported(AVX)) {
3563-
CpuFeatureScope scope(this, AVX);
3564-
vpmuludq(dst.fp(), lhs.fp(), rhs.fp());
3565-
} else {
3566-
if (dst.fp() != lhs.fp()) movaps(dst.fp(), lhs.fp());
3567-
pmuludq(dst.fp(), rhs.fp());
3568-
}
3562+
liftoff::EmitSimdCommutativeBinOp<&Assembler::vpmuludq, &Assembler::pmuludq>(
3563+
this, dst, lhs, rhs);
35693564
Paddq(dst.fp(), tmp2.fp());
35703565
}
35713566

‎deps/v8/test/mjsunit/regress/regress-1194869.js‎

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
// Copyright 2021 the V8 project authors. All rights reserved.
2+
// Use of this source code is governed by a BSD-style license that can be
3+
// found in the LICENSE file.
4+
//
5+
// Flags: --allow-natives-syntax --no-lazy-feedback-allocation
6+
// Flags: --interrupt-budget=100
7+
8+
function f() {
9+
return "".indexOf("", 2);
10+
}
11+
12+
%PrepareFunctionForOptimization(f)
13+
assertEquals(f(), 0);
14+
assertEquals(f(), 0);
15+
%OptimizeFunctionOnNextCall(f)
16+
assertEquals(f(), 0);
17+
assertEquals(f(), 0);
18+
19+
function g() {
20+
return "".indexOf("", 2);
21+
}
22+
for (let i = 0; i < 191; i++) {
23+
// Expect a natural optimization here due to low interrupt budget.
24+
assertEquals(g(), 0);
25+
}

0 commit comments

Comments
 (0)
Back | FazBrowse Home | New Git URL