Skip to content

Commit afbb6e4

Browse files
committed
deps: V8: cherry-pick 87356585659b
Original commit message: [flags] Ignore memory tuning flags when computing hash for code caching Change-Id: Ia9aa5b1da8d9b1356bea281cde91bca9ea0b4071 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/6938261 Reviewed-by: Leszek Swirski <[email protected]> Commit-Queue: Joyee Cheung <[email protected]> Cr-Commit-Position: refs/heads/main@{#102443} Refs: v8/v8@8735658
1 parent c6316f9 commit afbb6e4

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

common.gypi

Lines changed: 1 addition & 1 deletion
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.21',
41+
'v8_embedder_string': '-node.22',
4242

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

deps/v8/src/flags/flags.cc

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -473,6 +473,15 @@ uint32_t ComputeFlagListHash() {
473473
if (flag.PointsTo(&v8_flags.random_seed)) continue;
474474
if (flag.PointsTo(&v8_flags.predictable)) continue;
475475

476+
// These flags are not relevant for code caching and are often set by
477+
// embedders to tune memory usage.
478+
if (flag.PointsTo(&v8_flags.max_old_space_size) ||
479+
flag.PointsTo(&v8_flags.min_semi_space_size) ||
480+
flag.PointsTo(&v8_flags.max_semi_space_size) ||
481+
flag.PointsTo(&v8_flags.max_heap_size)) {
482+
continue;
483+
}
484+
476485
// The following flags are implied by --predictable (some negated).
477486
if (flag.PointsTo(&v8_flags.concurrent_sparkplug) ||
478487
flag.PointsTo(&v8_flags.concurrent_recompilation) ||

0 commit comments

Comments
 (0)