Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Commit 80907c0

Browse filesBrowse files
joyeecheungRafaelGSS
joyeecheung
authored and
RafaelGSS
committed
deps: V8: cherry-pick edeb0a4fa181
Original commit message: Zero-initialize proxy padding So that snapshots with proxies can be reproducible. Refs: #61898 Change-Id: I01fac5e18c73cd482a1ae63750dbadf42a12e08a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/7666243 Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Commit-Queue: Joyee Cheung <joyee@igalia.com> Cr-Commit-Position: refs/heads/main@{#105830} Refs: v8/v8@edeb0a4 PR-URL: #61898 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com> (cherry picked from commit 38d98c3)
1 parent 5e0dc16 commit 80907c0
Copy full SHA for 80907c0

3 files changed

+8-1Lines changed: 8 additions & 1 deletion

File tree

Expand file treeCollapse file tree
Open diff view settings
Filter options
Expand file treeCollapse file tree
Open diff view settings
Collapse file

‎common.gypi‎

Copy file name to clipboardExpand all lines: common.gypi
+1-1Lines 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.8',
41+
'v8_embedder_string': '-node.9',
4242

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

Collapse file

‎deps/v8/src/builtins/builtins-proxy-gen.cc‎

Copy file name to clipboardExpand all lines: deps/v8/src/builtins/builtins-proxy-gen.cc
+4Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,10 @@ TNode<JSProxy> ProxiesCodeStubAssembler::AllocateProxy(
6363
StoreObjectFieldNoWriteBarrier(proxy, JSProxy::kTargetOffset, target);
6464
StoreObjectFieldNoWriteBarrier(proxy, JSProxy::kHandlerOffset, handler);
6565
StoreObjectFieldNoWriteBarrier(proxy, JSProxy::kFlagsOffset, flags);
66+
#if TAGGED_SIZE_8_BYTES
67+
StoreObjectFieldNoWriteBarrier(proxy, JSProxy::kPaddingOffset,
68+
Int32Constant(0));
69+
#endif
6670

6771
return CAST(proxy);
6872
}
Collapse file

‎deps/v8/src/heap/factory.cc‎

Copy file name to clipboardExpand all lines: deps/v8/src/heap/factory.cc
+3Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3946,6 +3946,9 @@ Handle<JSProxy> Factory::NewJSProxy(DirectHandle<JSReceiver> target,
39463946
result->set_target(*target, SKIP_WRITE_BARRIER);
39473947
result->set_handler(*handler, SKIP_WRITE_BARRIER);
39483948
result->set_flags(JSProxy::IsRevocableBit::encode(revocable));
3949+
#if TAGGED_SIZE_8_BYTES
3950+
result->set_padding(0);
3951+
#endif
39493952
return handle(result, isolate());
39503953
}
39513954

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.