Mark DynamicallyAccessedMemberTypes.All as EditorBrosable(Never) (#117720)

Resolves #114197.

`DynamicallyAccessedMemberTypes.All` should be the least used member of this enum, but it is the most popular on all of GitHub. Use of it leads to spurious trimming warnings (because it makes it appear more things are reflected on than needed) and also too much extra size left on the table (for the same reason). Make it harder to discover.

Author: MichalStrehovsky

src/libraries/System.Private.CoreLib/src/System/Diagnostics/CodeAnalysis/DynamicallyAccessedMemberTypes.cs

@@ -1,6 +1,8 @@
 // Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the MIT license.
 
+using System.ComponentModel;
+
 namespace System.Diagnostics.CodeAnalysis
 {
     /// <summary>
@@ -165,6 +167,7 @@ enum DynamicallyAccessedMemberTypes
         /// <summary>
         /// Specifies all members.
         /// </summary>
+        [EditorBrowsable(EditorBrowsableState.Never)]
         All = ~None
     }
 }

src/libraries/System.Runtime/ref/System.Runtime.cs

@@ -8894,6 +8894,7 @@ public DynamicallyAccessedMembersAttribute(System.Diagnostics.CodeAnalysis.Dynam
     [System.FlagsAttribute]
     public enum DynamicallyAccessedMemberTypes
     {
+        [System.ComponentModel.EditorBrowsableAttribute(System.ComponentModel.EditorBrowsableState.Never)]
         All = -1,
         None = 0,
         PublicParameterlessConstructor = 1,