[cleanup] Delete test-only CppHeapPointerTable method [v8/v8 : main]
0 views
Skip to first unread message
Andreas Haas (Gerrit)
Oct 14, 2025, 12:48:44 PMOct 14
to Michael Lippautz, V8 LUCI CQ, AyeAye, oilpan-r...@chromium.org
Attention needed from Michael Lippautz
Andreas Haas added 1 comment![]( "Open in Gerrit")
Patchset-level comments
Related details
Attention is currently required from:
- Michael Lippautz
Submit Requirements:
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Michael Lippautz (Gerrit)
Oct 15, 2025, 3:47:09 AMOct 15
to Andreas Haas, V8 LUCI CQ, AyeAye, oilpan-r...@chromium.org
Attention needed from Andreas Haas
Michael Lippautz voted Code-Review+1![]( "Open in Gerrit")
Attention is currently required from:
- Andreas Haas
Submit Requirements:
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Andreas Haas (Gerrit)
Oct 15, 2025, 4:31:56 AMOct 15
to Michael Lippautz, V8 LUCI CQ, AyeAye, oilpan-r...@chromium.org
Andreas Haas voted Commit-Queue+2![]( "Open in Gerrit")
Submit Requirements:
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
V8 LUCI CQ (Gerrit)
Oct 15, 2025, 4:36:13 AMOct 15
to Andreas Haas, Michael Lippautz, AyeAye, oilpan-r...@chromium.org
V8 LUCI CQ submitted the change![]( "Open in Gerrit")
Change information
Commit message:
[cleanup] Delete test-only CppHeapPointerTable method
The method WriteLazilyInitializedCppHeapPointerField<> got obsolete
during a recent refactoring, and since then was only called in tests.
With this CL, the new version of
WriteLazilyInitializedCppHeapPointerField is used in the tests, and the
obsolete version gets deleted.
Change-Id: I79290034ea3cdf9bb360c843ad3767bd6b1861b3
Reviewed-by: Michael Lippautz <mlip...@chromium.org>
Commit-Queue: Andreas Haas <ah...@chromium.org>
Cr-Commit-Position: refs/heads/main@{#103124}
Files:
- M src/objects/cpp-heap-object-wrapper-inl.h
- M src/objects/cpp-heap-object-wrapper.h
- M src/objects/heap-object.h
- M src/objects/objects-inl.h
- M src/sandbox/cppheap-pointer-inl.h
- M src/sandbox/cppheap-pointer.h
- M test/unittests/heap/cppgc-js/unified-heap-utils.cc
Change size: M
Delta: 7 files changed, 4 insertions(+), 53 deletions(-)
Branch: refs/heads/main
Submit Requirements:
Code-Review: +1 by Michael Lippautz
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages