From 9814beb66e232e50c04b975d7f41ccfa662d5254 Mon Sep 17 00:00:00 2001
From: Eric Erhardt <eric.erhardt@microsoft.com>
Date: Fri, 10 Jul 2020 02:04:36 -0500
Subject: [PATCH] Define Debugger.IsSupported feature switch to allow trimming
 of debugger only code. (#37288)

---
 src/libraries/Microsoft.CSharp/src/ILLinkTrim.xml       |  9 ++++++---
 src/libraries/System.Linq/src/ILLinkTrim.xml            |  4 ++--
 .../src/ILLink/ILLink.Descriptors.Shared.xml            | 17 ++++++++++-------
 3 files changed, 18 insertions(+), 12 deletions(-)

diff --git a/src/libraries/Microsoft.CSharp/src/ILLinkTrim.xml b/src/libraries/Microsoft.CSharp/src/ILLinkTrim.xml
index 4260df0..25a8429 100644
--- a/src/libraries/Microsoft.CSharp/src/ILLinkTrim.xml
+++ b/src/libraries/Microsoft.CSharp/src/ILLinkTrim.xml
@@ -1,10 +1,13 @@
 <linker>
   <assembly fullname="Microsoft.CSharp">
+    <!-- Required for COM event dispatch -->
+    <type fullname="System.Runtime.InteropServices.ComEventsSink"/>
+  </assembly>
+
+  <assembly fullname="Microsoft.CSharp" feature="System.Diagnostics.Debugger.IsSupported" featurevalue="true" featuredefault="true">
     <!-- Required by the Expression Evaluator -->
     <type fullname="Microsoft.CSharp.RuntimeBinder.DynamicMetaObjectProviderDebugView"/>
     <!-- Required by the Expression Evaluator -->
     <type fullname="Microsoft.CSharp.RuntimeBinder.DynamicMetaObjectProviderDebugView/DynamicDebugViewEmptyException"/>
-    <!-- Required for COM event dispatch -->
-    <type fullname="System.Runtime.InteropServices.ComEventsSink"/>
   </assembly>
-</linker>
\ No newline at end of file
+</linker>
diff --git a/src/libraries/System.Linq/src/ILLinkTrim.xml b/src/libraries/System.Linq/src/ILLinkTrim.xml
index a2e3201..a24078d 100644
--- a/src/libraries/System.Linq/src/ILLinkTrim.xml
+++ b/src/libraries/System.Linq/src/ILLinkTrim.xml
@@ -1,8 +1,8 @@
 <linker>
-  <assembly fullname="System.Linq">
+  <assembly fullname="System.Linq" feature="System.Diagnostics.Debugger.IsSupported" featurevalue="true" featuredefault="true">
     <!-- required by debugger, see comment in System/Linq/DebugView.cs -->
     <type fullname="System.Linq.SystemCore_EnumerableDebugView" />
     <type fullname="System.Linq.SystemCore_EnumerableDebugView`1" />
     <type fullname="System.Linq.SystemCore_EnumerableDebugViewEmptyException" />
   </assembly>
-</linker>
\ No newline at end of file
+</linker>
diff --git a/src/libraries/System.Private.CoreLib/src/ILLink/ILLink.Descriptors.Shared.xml b/src/libraries/System.Private.CoreLib/src/ILLink/ILLink.Descriptors.Shared.xml
index aa0a42d..71c7c46 100644
--- a/src/libraries/System.Private.CoreLib/src/ILLink/ILLink.Descriptors.Shared.xml
+++ b/src/libraries/System.Private.CoreLib/src/ILLink/ILLink.Descriptors.Shared.xml
@@ -4,7 +4,16 @@
       <!-- Internal API used by tests only. -->
       <method name="GetICUVersion" />
     </type>
-    <!-- Properties and methods used by a debugger. -->
+    <type fullname="System.Threading.ThreadPoolBoundHandle">
+      <!-- Workaround to keep .interfaceimpl even though this type
+             is not instantiated on unix:
+             https://github.com/mono/linker/pull/649 -->
+      <method name=".ctor" />
+    </type>
+  </assembly>
+  
+  <!-- Properties and methods used by a debugger. -->
+  <assembly fullname="System.Private.CoreLib" feature="System.Diagnostics.Debugger.IsSupported" featurevalue="true" featuredefault="true">
     <type fullname="System.Threading.Tasks.Task">
       <property name="ParentForDebugger" />
       <property name="StateFlagsForDebugger" />
@@ -37,11 +46,5 @@
       <!-- Methods is used by VS Tasks Window. -->
       <method name="GetActiveTaskFromId" />
     </type>
-    <type fullname="System.Threading.ThreadPoolBoundHandle">
-      <!-- Workaround to keep .interfaceimpl even though this type
-             is not instantiated on unix:
-             https://github.com/mono/linker/pull/649 -->
-      <method name=".ctor" />
-    </type>
   </assembly>
 </linker>
-- 
2.7.4