Skip to content

Commit 577eb44

Browse files
Apply suggestions from code review
Co-authored-by: Noah Falk <[email protected]>
1 parent 981ab62 commit 577eb44

File tree

1 file changed

+1
-6
lines changed

1 file changed

+1
-6
lines changed

src/coreclr/vm/ceeload.cpp

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1975,19 +1975,14 @@ void Module::FreeClassTables()
19751975
{
19761976
TypeHandle th = pEntry->GetTypeHandle();
19771977

1978-
// If class is not fully loaded there is nothing to destroy or notify about
1979-
if (!th.IsRestored())
1980-
continue;
1981-
1982-
// Type desc is handled differently
1978+
// Array EEClass doesn't need notification and there is no work for Destruct()
19831979
if (th.IsTypeDesc())
19841980
continue;
19851981

19861982
MethodTable * pMT = th.AsMethodTable();
19871983
EEClass::NotifyUnload(pMT, true);
19881984

19891985
// We need to call destruct on instances of EEClass whose "canonical" dependent lives in this table
1990-
// There is nothing interesting to destruct on array EEClass
19911986
if (pMT->IsCanonicalMethodTable())
19921987
{
19931988
pMT->GetClass()->Destruct(pMT);

0 commit comments

Comments
 (0)