Commit 91b3bd3
authored
deps: patch V8 to 13.7.152.13
Refs: v8/v8@13.7.152.10...13.7.152.13
PR-URL: #58539
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>1 parent 790acc8 commit 91b3bd3
4 files changed
Lines changed: 25 additions & 11 deletions
File tree
- deps/v8
- include
- infra/mb
- src/compiler
- turboshaft
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -11,7 +11,7 @@ | |||
| 11 | 11 | #define V8_MAJOR_VERSION 13 | |
| 12 | 12 | #define V8_MINOR_VERSION 7 | |
| 13 | 13 | #define V8_BUILD_NUMBER 152 | |
| 14 | - #define V8_PATCH_LEVEL 10 | ||
| 14 | + #define V8_PATCH_LEVEL 13 | ||
| 15 | 15 | ||
| 16 | 16 | // Use 1 for candidates and 0 otherwise. | |
| 17 | 17 | // (Boolean macro values are not supported by all preprocessors.) | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -953,7 +953,7 @@ | |||
| 953 | 953 | }, | |
| 954 | 954 | ||
| 955 | 955 | 'no_reclient': { | |
| 956 | - 'gn_args': 'use_remoteexec=false', | ||
| 956 | + 'gn_args': 'use_remoteexec=false use_siso=false', | ||
| 957 | 957 | }, | |
| 958 | 958 | ||
| 959 | 959 | 'no_sandbox': { | |
@@ -968,8 +968,11 @@ | |||
| 968 | 968 | 'gn_args': 'v8_use_perfetto=true', | |
| 969 | 969 | }, | |
| 970 | 970 | ||
| 971 | + # TODO(https://crbug.com/414724525): Temporarily use the reclient and siso | ||
| 972 | + # configs synonym. In a follow up we'll drop the reclient parts and replace | ||
| 973 | + # them with SISO configs. | ||
| 971 | 974 | 'reclient': { | |
| 972 | - 'gn_args': 'use_remoteexec=true', | ||
| 975 | + 'gn_args': 'use_remoteexec=true use_siso=true', | ||
| 973 | 976 | }, | |
| 974 | 977 | ||
| 975 | 978 | 'release': { | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1344,7 +1344,12 @@ Node* RepresentationChanger::GetWord64RepresentationFor( | |||
| 1344 | 1344 | } | |
| 1345 | 1345 | } else if (output_rep == MachineRepresentation::kTaggedSigned) { | |
| 1346 | 1346 | if (output_type.Is(Type::SignedSmall())) { | |
| 1347 | - op = simplified()->ChangeTaggedSignedToInt64(); | ||
| 1347 | + if (output_type.IsRange() && output_type.AsRange()->Min() >= 0) { | ||
| 1348 | + node = InsertChangeTaggedSignedToInt32(node); | ||
| 1349 | + op = machine()->ChangeUint32ToUint64(); | ||
| 1350 | + } else { | ||
| 1351 | + op = simplified()->ChangeTaggedSignedToInt64(); | ||
| 1352 | + } | ||
| 1348 | 1353 | } else { | |
| 1349 | 1354 | return TypeError(node, output_rep, output_type, | |
| 1350 | 1355 | MachineRepresentation::kWord64); | |
Lines changed: 13 additions & 7 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -325,10 +325,11 @@ class RedundantStoreAnalysis { | |||
| 325 | 325 | // TODO(nicohartmann@): Use the new effect flags to distinguish heap | |
| 326 | 326 | // access once available. | |
| 327 | 327 | const bool is_on_heap_store = store.kind.tagged_base; | |
| 328 | - const bool is_field_store = !store.index().valid(); | ||
| 328 | + const bool is_fixed_offset_store = !store.index().valid(); | ||
| 329 | 329 | const uint8_t size = store.stored_rep.SizeInBytes(); | |
| 330 | - // For now we consider only stores of fields of objects on the heap. | ||
| 331 | - if (is_on_heap_store && is_field_store) { | ||
| 330 | + // For now we consider only stores of fixed offsets of objects on the | ||
| 331 | + // heap. | ||
| 332 | + if (is_on_heap_store && is_fixed_offset_store) { | ||
| 332 | 333 | bool is_eliminable_store = false; | |
| 333 | 334 | switch (table_.GetObservability(store.base(), store.offset, size)) { | |
| 334 | 335 | case StoreObservability::kUnobservable: | |
@@ -415,11 +416,16 @@ class RedundantStoreAnalysis { | |||
| 415 | 416 | // TODO(nicohartmann@): Use the new effect flags to distinguish heap | |
| 416 | 417 | // access once available. | |
| 417 | 418 | const bool is_on_heap_load = load.kind.tagged_base; | |
| 418 | - const bool is_field_load = !load.index().valid(); | ||
| 419 | + const bool is_fixed_offset_load = !load.index().valid(); | ||
| 419 | 420 | // For now we consider only loads of fields of objects on the heap. | |
| 420 | - if (is_on_heap_load && is_field_load) { | ||
| 421 | - table_.MarkPotentiallyAliasingStoresAsObservable(load.base(), | ||
| 422 | - load.offset); | ||
| 421 | + if (is_on_heap_load) { | ||
| 422 | + if (is_fixed_offset_load) { | ||
| 423 | + table_.MarkPotentiallyAliasingStoresAsObservable(load.base(), | ||
| 424 | + load.offset); | ||
| 425 | + } else { | ||
| 426 | + // A dynamically indexed load might alias any fixed offset. | ||
| 427 | + table_.MarkAllStoresAsObservable(); | ||
| 428 | + } | ||
| 423 | 429 | } | |
| 424 | 430 | break; | |
| 425 | 431 | } | |
0 commit comments