Skip to content

Commit 67e3591

Browse files
rmacnak-googleCommit Queue
authored andcommitted
[vm] Remove dead Isolate::boxed_field_list_.
TEST=ci Change-Id: I072ab19a8195c5b1cf1b32a6541a31a25bc8a1fc Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/439343 Reviewed-by: Alexander Aprelev <[email protected]> Commit-Queue: Ryan Macnak <[email protected]>
1 parent 4115083 commit 67e3591

File tree

2 files changed

+0
-17
lines changed

2 files changed

+0
-17
lines changed

runtime/vm/isolate.cc

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -355,8 +355,6 @@ IsolateGroup::IsolateGroup(std::shared_ptr<IsolateGroupSource> source,
355355
kernel_data_class_cache_mutex_(),
356356
kernel_constants_mutex_(),
357357
shared_field_initializer_rwlock_(),
358-
field_list_mutex_(),
359-
boxed_field_list_(GrowableObjectArray::null()),
360358
program_lock_(new SafepointRwLock(SafepointLevel::kGCAndDeopt)),
361359
active_mutators_monitor_(new Monitor()),
362360
max_active_mutators_(Scavenger::MaxMutatorThreadCount()),
@@ -2968,13 +2966,6 @@ void IsolateGroup::VisitSharedPointers(ObjectPointerVisitor* visitor,
29682966
case kSharedFieldTable:
29692967
shared_field_table()->VisitObjectPointers(visitor);
29702968
break;
2971-
case kBoxedFieldList:
2972-
// Visit the boxed_field_list_.
2973-
// 'boxed_field_list_' access via mutator and background compilation
2974-
// threads is guarded with a monitor. This means that we can visit it only
2975-
// when at safepoint or the field_list_mutex_ lock has been taken.
2976-
visitor->VisitPointer(reinterpret_cast<ObjectPtr*>(&boxed_field_list_));
2977-
break;
29782969
case kBackgroundCompiler:
29792970
NOT_IN_PRECOMPILED(background_compiler()->VisitPointers(visitor));
29802971
break;

runtime/vm/isolate.h

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -271,7 +271,6 @@ enum RootSlice : intptr_t {
271271
kSentinelFieldTable,
272272
kSharedInitialFieldTable,
273273
kSharedFieldTable,
274-
kBoxedFieldList,
275274
kBackgroundCompiler,
276275
kDebugger,
277276
kReloadContext,
@@ -300,8 +299,6 @@ inline const char* RootSliceToCString(intptr_t slice) {
300299
return "shared initial field table";
301300
case kSharedFieldTable:
302301
return "shared field table";
303-
case kBoxedFieldList:
304-
return "boxed field list";
305302
case kBackgroundCompiler:
306303
return "background compiler";
307304
case kDebugger:
@@ -984,11 +981,6 @@ class IsolateGroup : public IntrusiveDListEntry<IsolateGroup> {
984981
// Ensure exclusive execution of shared field initializers.
985982
SafepointRwLock shared_field_initializer_rwlock_;
986983

987-
// Protect access to boxed_field_list_.
988-
Mutex field_list_mutex_;
989-
// List of fields that became boxed and that trigger deoptimization.
990-
GrowableObjectArrayPtr boxed_field_list_;
991-
992984
// Ensures synchronized access to classes functions, fields and other
993985
// program structure elements to accommodate concurrent modification done
994986
// by multiple isolates and background compiler.

0 commit comments

Comments
 (0)