Remove ComputeSuppressUnmanagedCodeAccessAttr() and usage. (dotnet/coreclr#26409)
authorAaron Robinson <arobins@microsoft.com>
Wed, 28 Aug 2019 03:20:58 +0000 (20:20 -0700)
committerJan Kotas <jkotas@microsoft.com>
Wed, 28 Aug 2019 03:20:58 +0000 (20:20 -0700)
Commit migrated from https://github.com/dotnet/coreclr/commit/7f8014fc2ba126149a361c9de21e5f9ee1fca7cf

src/coreclr/src/vm/clrtocomcall.cpp
src/coreclr/src/vm/method.cpp
src/coreclr/src/vm/method.hpp
src/coreclr/src/vm/methodtablebuilder.cpp

index d111d1b..eba7615 100644 (file)
@@ -146,8 +146,6 @@ ComPlusCallInfo *ComPlusCall::PopulateComPlusCallMethodDesc(MethodDesc* pMD, DWO
     if (pdwStubFlags == NULL)
         return pComInfo;
 
-    pMD->ComputeSuppressUnmanagedCodeAccessAttr(pMD->GetMDImport());
-
     //
     // Compute NDirectStubFlags
     //
index 5de254e..fd0d13f 100644 (file)
@@ -5325,19 +5325,6 @@ void NDirectMethodDesc::InitEarlyBoundNDirectTarget()
 #endif // !CROSSGEN_COMPILE
 
 //*******************************************************************************
-void MethodDesc::ComputeSuppressUnmanagedCodeAccessAttr(IMDInternalImport *pImport)
-{
-    CONTRACTL
-    {
-        THROWS;
-        GC_NOTRIGGER;
-        FORBID_FAULT;
-    }
-    CONTRACTL_END;
-
-}
-
-//*******************************************************************************
 BOOL MethodDesc::HasNativeCallableAttribute()
 {
 
index 6bbaea7..596316b 100644 (file)
@@ -668,7 +668,6 @@ public:
         return GetMethodTable()->IsInterface();
     }
 
-    void ComputeSuppressUnmanagedCodeAccessAttr(IMDInternalImport *pImport);
     BOOL HasNativeCallableAttribute();
 
 #ifdef FEATURE_COMINTEROP 
index d34f068..a135ff5 100644 (file)
@@ -6047,11 +6047,6 @@ MethodTableBuilder::InitMethodDesc(
     if (IsMdStatic(dwMemberAttrs))
         pNewMD->SetStatic();
 
-    // Set suppress unmanaged code access permission attribute
-
-    if (pNewMD->IsNDirect())
-        pNewMD->ComputeSuppressUnmanagedCodeAccessAttr(pIMDII);
-
 #ifdef _DEBUG 
     // Mark as many methods as synchronized as possible.
     //