Fix AssemblyName cache hash and key (#24138)

* Add ContextualReflection LoadWithPartialName case

* Remove unnecessary MethodImplOptions.NoInlining

* Remove m_assembly warning

* Fix AssemblyName hash function

* AssemblyNative::Load fix stackMark usage

Do not use the stackMark if (ptrLoadContextBinder != NULL)

* Temporarily disable DefaultContextOverrideTPA

Test is failing due to a logic error.
Fix is pending in https://github.com/dotnet/corefx/pull/37071

1 files changed, 5 insertions, 5 deletions

diff --git a/src/vm/assemblynative.cpp b/src/vm/assemblynative.cpp
index aa830932ed..cc95151408 100644
--- a/src/vm/assemblynative.cpp
+++ b/src/vm/assemblynative.cpp
@@ -78,18 +78,18 @@ FCIMPL6(Object*, AssemblyNative::Load, AssemblyNameBaseObject* assemblyNameUNSAF
     else
     {
         // Compute parent assembly
-        if (gc.requestingAssembly == NULL)
+        if (gc.requestingAssembly != NULL)
         {
-            pRefAssembly = SystemDomain::GetCallersAssembly(stackMark);
+            pRefAssembly = gc.requestingAssembly->GetAssembly();
         }
-        else
+        else if (ptrLoadContextBinder == NULL)
         {
-            pRefAssembly = gc.requestingAssembly->GetAssembly();
+            pRefAssembly = SystemDomain::GetCallersAssembly(stackMark);
         }
         
         if (pRefAssembly)
         {
-            pParentAssembly= pRefAssembly->GetDomainAssembly();
+            pParentAssembly = pRefAssembly->GetDomainAssembly();
         }
     }