diff options
author | Jan Vorlicek <janvorli@microsoft.com> | 2018-11-28 06:36:59 +0100 |
---|---|---|
committer | Jan Kotas <jkotas@microsoft.com> | 2018-11-27 21:36:59 -0800 |
commit | 8aa0869eb9153429091fdba49469d89ec33092cb (patch) | |
tree | 8d9caeb728ba245c9e311cd411e59aab9a21d30f /src/vm/appdomain.cpp | |
parent | c7c166e33c807d938367d0364850fe7af1c7b835 (diff) | |
download | coreclr-8aa0869eb9153429091fdba49469d89ec33092cb.tar.gz coreclr-8aa0869eb9153429091fdba49469d89ec33092cb.tar.bz2 coreclr-8aa0869eb9153429091fdba49469d89ec33092cb.zip |
Fix ILStubCache allocation for collectible assemblies (#21188)
The ILStubCache was being allocated per domain unless the domain was a
compilation AppDomain. This is wrong for collectible assemblies, since
after an assembly is collected, the cache keeps stale entries referring
to already deleted MethodTables.
The fix is to make ILStubChange per LoaderAllocator instead (and keep
the per module instances for compilation AppDomain).
Diffstat (limited to 'src/vm/appdomain.cpp')
-rw-r--r-- | src/vm/appdomain.cpp | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/vm/appdomain.cpp b/src/vm/appdomain.cpp index ce57ee0c5f..3b4f55ae72 100644 --- a/src/vm/appdomain.cpp +++ b/src/vm/appdomain.cpp @@ -3840,9 +3840,6 @@ void AppDomain::Init() BaseDomain::Init(); - // Set up the IL stub cache - m_ILStubCache.Init(GetLoaderAllocator()->GetHighFrequencyHeap()); - // Set up the binding caches m_AssemblyCache.Init(&m_DomainCacheCrst, GetHighFrequencyHeap()); m_UnmanagedCache.InitializeTable(this, &m_DomainCacheCrst); |