Commit d3eabf79 authored by Steve Blackburn's avatar Steve Blackburn Committed by Commit Bot

TPH no longer uses V8 heap.

Simple correction over prior commit since TPH no longer needs
to depend on V8 read only heap.

Bug: v8:9533
Change-Id: Icc72cdbb58fb3d9dc79573548cd9fe9cc88b2684
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2039350Reviewed-by: 's avatarUlan Degenbaev <ulan@chromium.org>
Commit-Queue: Steve Blackburn <steveblackburn@google.com>
Cr-Commit-Position: refs/heads/master@{#66151}
parent e4a4c81e
......@@ -144,9 +144,7 @@ bool ReadOnlyHeap::Contains(Address address) {
// static
bool ReadOnlyHeap::Contains(HeapObject object) {
if (V8_ENABLE_THIRD_PARTY_HEAP_BOOL) {
// read only includes both TPH and the snapshot, so need both checks
return third_party_heap::Heap::InReadOnlySpace(object.address()) ||
MemoryChunk::FromHeapObject(object)->InReadOnlySpace();
return third_party_heap::Heap::InReadOnlySpace(object.address());
} else {
return MemoryChunk::FromHeapObject(object)->InReadOnlySpace();
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment