Commit df2f1ad
deps: V8: cherry-pick f915fa4c9f41
Original commit message:
[osr] Ensure trying to osr does not skip loop interrupts
Fixed: 374013413
Change-Id: I52d7b4e165e0abd0bd517a81d2e8ef3f1f802bfb
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/5946288
Commit-Queue: Darius Mercadier <dmercadier@chromium.org>
Auto-Submit: Olivier Flückiger <olivf@chromium.org>
Reviewed-by: Darius Mercadier <dmercadier@chromium.org>
Cr-Commit-Position: refs/heads/main@{#96708}
Refs: v8/v8@f915fa4
PR-URL: #55484
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent 37352ce commit df2f1ad
3 files changed
Lines changed: 24 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -36,7 +36,7 @@ | |||
| 36 | 36 | ||
| 37 | 37 | # Reset this number to 0 on major V8 upgrades. | |
| 38 | 38 | # Increment by one for each non-official patch applied to deps/v8. | |
| 39 | - 'v8_embedder_string': '-node.10', | ||
| 39 | + 'v8_embedder_string': '-node.11', | ||
| 40 | 40 | ||
| 41 | 41 | ##### V8 defaults for Node.js ##### | |
| 42 | 42 | ||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1338,7 +1338,14 @@ MaybeHandle<Code> GetOrCompileOptimized( | |||
| 1338 | 1338 | } | |
| 1339 | 1339 | ||
| 1340 | 1340 | // Do not optimize when debugger needs to hook into every call. | |
| 1341 | - if (isolate->debug()->needs_check_on_function_call()) return {}; | ||
| 1341 | + if (isolate->debug()->needs_check_on_function_call()) { | ||
| 1342 | + // Reset the OSR urgency to avoid triggering this compilation request on | ||
| 1343 | + // every iteration and thereby skipping other interrupts. | ||
| 1344 | + if (IsOSR(osr_offset)) { | ||
| 1345 | + function->feedback_vector()->reset_osr_urgency(); | ||
| 1346 | + } | ||
| 1347 | + return {}; | ||
| 1348 | + } | ||
| 1342 | 1349 | ||
| 1343 | 1350 | // Do not optimize if we need to be able to set break points. | |
| 1344 | 1351 | if (shared->HasBreakInfo(isolate)) return {}; | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -0,0 +1,15 @@ | |||
| 1 | + // Copyright 2024 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: --enable-inspector | ||
| 6 | + | ||
| 7 | + var Debug = debug.Debug; | ||
| 8 | + Debug.sendMessageForMethodChecked('Runtime.enable', {}); | ||
| 9 | + const {msgid, msg} = Debug.createMessage('Runtime.evaluate', { | ||
| 10 | + expression: 'while(true) {}', | ||
| 11 | + throwOnSideEffect: true, | ||
| 12 | + timeout: 1000, | ||
| 13 | + }) | ||
| 14 | + Debug.sendMessage(msg); | ||
| 15 | + Debug.takeReplyChecked(msgid).toString(); | ||
0 commit comments