diff options
author | Filip Navara <filip.navara@gmail.com> | 2019-01-03 14:46:57 +0100 |
---|---|---|
committer | Jan Kotas <jkotas@microsoft.com> | 2019-01-03 03:46:57 -1000 |
commit | 6f9121bf9e51472780e25d7f50b8c9e46898ad50 (patch) | |
tree | a389f61389023d488a0b4a6d6b09ccfc79aa6d0b /src/vm | |
parent | acbfc3d9b0729cc5c2957d5536a894c07b1c4af8 (diff) | |
download | coreclr-6f9121bf9e51472780e25d7f50b8c9e46898ad50.tar.gz coreclr-6f9121bf9e51472780e25d7f50b8c9e46898ad50.tar.bz2 coreclr-6f9121bf9e51472780e25d7f50b8c9e46898ad50.zip |
Remove unnecessary propagation of stack crawl marks. (#21781)
Diffstat (limited to 'src/vm')
-rw-r--r-- | src/vm/appdomain.cpp | 3 | ||||
-rw-r--r-- | src/vm/appdomain.hpp | 1 | ||||
-rw-r--r-- | src/vm/assemblynative.cpp | 2 | ||||
-rw-r--r-- | src/vm/assemblyspec.cpp | 9 | ||||
-rw-r--r-- | src/vm/assemblyspec.hpp | 9 | ||||
-rw-r--r-- | src/vm/compile.cpp | 2 | ||||
-rw-r--r-- | src/vm/compile.h | 1 | ||||
-rw-r--r-- | src/vm/coreassemblyspec.cpp | 3 |
8 files changed, 10 insertions, 20 deletions
diff --git a/src/vm/appdomain.cpp b/src/vm/appdomain.cpp index fe0c694462..19035604d6 100644 --- a/src/vm/appdomain.cpp +++ b/src/vm/appdomain.cpp @@ -6068,7 +6068,6 @@ AppDomain::BindHostedPrivAssembly( PEAssembly * AppDomain::BindAssemblySpec( AssemblySpec * pSpec, BOOL fThrowOnFileNotFound, - StackCrawlMark * pCallerStackMark, BOOL fUseHostBinderIfAvailable) { STATIC_CONTRACT_THROWS; @@ -6169,7 +6168,7 @@ EndTry2:; // Use CoreClr's fusion alternative CoreBindResult bindResult; - pSpec->Bind(this, fThrowOnFileNotFound, &bindResult, FALSE /* fNgenExplicitBind */, FALSE /* fExplicitBindToNativeImage */, pCallerStackMark); + pSpec->Bind(this, fThrowOnFileNotFound, &bindResult, FALSE /* fNgenExplicitBind */, FALSE /* fExplicitBindToNativeImage */); hrBindResult = bindResult.GetHRBindResult(); if (bindResult.Found()) diff --git a/src/vm/appdomain.hpp b/src/vm/appdomain.hpp index 0aeab8f1e4..9db5ad5584 100644 --- a/src/vm/appdomain.hpp +++ b/src/vm/appdomain.hpp @@ -2263,7 +2263,6 @@ public: virtual PEAssembly * BindAssemblySpec( AssemblySpec *pSpec, BOOL fThrowOnFileNotFound, - StackCrawlMark *pCallerStackMark = NULL, BOOL fUseHostBinderIfAvailable = TRUE) DAC_EMPTY_RET(NULL); HRESULT BindAssemblySpecForHostedBinder( diff --git a/src/vm/assemblynative.cpp b/src/vm/assemblynative.cpp index 14bef33c48..b291dbea8a 100644 --- a/src/vm/assemblynative.cpp +++ b/src/vm/assemblynative.cpp @@ -135,7 +135,7 @@ FCIMPL7(Object*, AssemblyNative::Load, AssemblyNameBaseObject* assemblyNameUNSAF { GCX_PREEMP(); - pAssembly = spec.LoadAssembly(FILE_LOADED, fThrowOnFileNotFound, stackMark); + pAssembly = spec.LoadAssembly(FILE_LOADED, fThrowOnFileNotFound); } if (pAssembly != NULL) diff --git a/src/vm/assemblyspec.cpp b/src/vm/assemblyspec.cpp index 136589fc48..b2d33b301c 100644 --- a/src/vm/assemblyspec.cpp +++ b/src/vm/assemblyspec.cpp @@ -753,7 +753,7 @@ PEAssembly *AssemblySpec::ResolveAssemblyFile(AppDomain *pDomain) } -Assembly *AssemblySpec::LoadAssembly(FileLoadLevel targetLevel, BOOL fThrowOnFileNotFound, StackCrawlMark *pCallerStackMark) +Assembly *AssemblySpec::LoadAssembly(FileLoadLevel targetLevel, BOOL fThrowOnFileNotFound) { CONTRACTL { @@ -763,7 +763,7 @@ Assembly *AssemblySpec::LoadAssembly(FileLoadLevel targetLevel, BOOL fThrowOnFil } CONTRACTL_END; - DomainAssembly * pDomainAssembly = LoadDomainAssembly(targetLevel, fThrowOnFileNotFound, pCallerStackMark); + DomainAssembly * pDomainAssembly = LoadDomainAssembly(targetLevel, fThrowOnFileNotFound); if (pDomainAssembly == NULL) { _ASSERTE(!fThrowOnFileNotFound); return NULL; @@ -891,8 +891,7 @@ ICLRPrivBinder* AssemblySpec::GetBindingContextFromParentAssembly(AppDomain *pDo } DomainAssembly *AssemblySpec::LoadDomainAssembly(FileLoadLevel targetLevel, - BOOL fThrowOnFileNotFound, - StackCrawlMark *pCallerStackMark) + BOOL fThrowOnFileNotFound) { CONTRACT(DomainAssembly *) { @@ -932,7 +931,7 @@ DomainAssembly *AssemblySpec::LoadDomainAssembly(FileLoadLevel targetLevel, } - PEAssemblyHolder pFile(pDomain->BindAssemblySpec(this, fThrowOnFileNotFound, pCallerStackMark)); + PEAssemblyHolder pFile(pDomain->BindAssemblySpec(this, fThrowOnFileNotFound)); if (pFile == NULL) RETURN NULL; diff --git a/src/vm/assemblyspec.hpp b/src/vm/assemblyspec.hpp index 5cf87f2c35..97bbd5faa7 100644 --- a/src/vm/assemblyspec.hpp +++ b/src/vm/assemblyspec.hpp @@ -216,15 +216,12 @@ class AssemblySpec : public BaseAssemblySpec BOOL fThrowOnFileNotFound, CoreBindResult* pBindResult, BOOL fNgenExplicitBind = FALSE, - BOOL fExplicitBindToNativeImage = FALSE, - StackCrawlMark *pCallerStackMark = NULL ); + BOOL fExplicitBindToNativeImage = FALSE); Assembly *LoadAssembly(FileLoadLevel targetLevel, - BOOL fThrowOnFileNotFound = TRUE, - StackCrawlMark *pCallerStackMark = NULL); + BOOL fThrowOnFileNotFound = TRUE); DomainAssembly *LoadDomainAssembly(FileLoadLevel targetLevel, - BOOL fThrowOnFileNotFound = TRUE, - StackCrawlMark *pCallerStackMark = NULL); + BOOL fThrowOnFileNotFound = TRUE); //**************************************************************************************** // diff --git a/src/vm/compile.cpp b/src/vm/compile.cpp index 3cec66e833..323c66abca 100644 --- a/src/vm/compile.cpp +++ b/src/vm/compile.cpp @@ -7221,7 +7221,6 @@ void ReportMissingDependency(Exception * e) PEAssembly *CompilationDomain::BindAssemblySpec( AssemblySpec *pSpec, BOOL fThrowOnFileNotFound, - StackCrawlMark *pCallerStackMark, BOOL fUseHostBinderIfAvailable) { PEAssembly *pFile = NULL; @@ -7238,7 +7237,6 @@ PEAssembly *CompilationDomain::BindAssemblySpec( pFile = AppDomain::BindAssemblySpec( pSpec, fThrowOnFileNotFound, - pCallerStackMark, fUseHostBinderIfAvailable); } EX_HOOK diff --git a/src/vm/compile.h b/src/vm/compile.h index 79991ddee6..e86cd07523 100644 --- a/src/vm/compile.h +++ b/src/vm/compile.h @@ -791,7 +791,6 @@ class CompilationDomain : public AppDomain, PEAssembly *BindAssemblySpec( AssemblySpec *pSpec, BOOL fThrowOnFileNotFound, - StackCrawlMark *pCallerStackMark = NULL, BOOL fUseHostBinderIfAvailable = TRUE) DAC_EMPTY_RET(NULL); BOOL CanEagerBindToZapFile(Module *targetModule, BOOL limitToHardBindList = TRUE); diff --git a/src/vm/coreassemblyspec.cpp b/src/vm/coreassemblyspec.cpp index 832b33eac0..44da97220e 100644 --- a/src/vm/coreassemblyspec.cpp +++ b/src/vm/coreassemblyspec.cpp @@ -107,8 +107,7 @@ VOID AssemblySpec::Bind(AppDomain *pAppDomain, BOOL fThrowOnFileNotFound, CoreBindResult *pResult, BOOL fNgenExplicitBind /* = FALSE */, - BOOL fExplicitBindToNativeImage /* = FALSE */, - StackCrawlMark *pCallerStackMark /* = NULL */) + BOOL fExplicitBindToNativeImage /* = FALSE */) { CONTRACTL { |