From fc3ae5a3cec383f1c2b048daa315de400d78af18 Mon Sep 17 00:00:00 2001
From: Eric Erhardt <eric.erhardt@microsoft.com>
Date: Mon, 1 Jun 2020 13:18:27 -0500
Subject: [PATCH] Define Debugger.IsSupported feature switch to allow trimming
 of debugger only code.

---
 .../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 4260df005ff0d1..25a8429f396d3f 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 a2e3201c32fcb2..a24078d42f5939 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 aa0a42d8f87aad..71c7c46c603215 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>