Remove two unnecessary ToArray calls (#22119)

We can just copy directly from the List, rather than first converting the list to an array and then copying that.

1 files changed, 6 insertions, 6 deletions

diff --git a/src/System.Private.CoreLib/src/System/Attribute.CoreCLR.cs b/src/System.Private.CoreLib/src/System/Attribute.CoreCLR.cs
index 94143f7005..bb8d2749fa 100644
--- a/src/System.Private.CoreLib/src/System/Attribute.CoreCLR.cs
+++ b/src/System.Private.CoreLib/src/System/Attribute.CoreCLR.cs
@@ -48,9 +48,9 @@ namespace System
                 AddAttributesToList(attributeList, attributes, types);
                 baseProp = GetParentDefinition(baseProp, indexParamTypes);
             }
-            Array array = CreateAttributeArrayHelper(type, attributeList.Count);
-            Array.Copy(attributeList.ToArray(), 0, array, 0, attributeList.Count);
-            return (Attribute[])array;
+            Attribute[] array = CreateAttributeArrayHelper(type, attributeList.Count);
+            attributeList.CopyTo(array, 0);
+            return array;
         }
 
         private static bool InternalIsDefined(PropertyInfo element, Type attributeType, bool inherit)
@@ -143,9 +143,9 @@ namespace System
                     AddAttributesToList(attributeList, attributes, types);
                     baseEvent = GetParentDefinition(baseEvent);
                 }
-                Array array = CreateAttributeArrayHelper(type, attributeList.Count);
-                Array.Copy(attributeList.ToArray(), 0, array, 0, attributeList.Count);
-                return (Attribute[])array;
+                Attribute[] array = CreateAttributeArrayHelper(type, attributeList.Count);
+                attributeList.CopyTo(array, 0);
+                return array;
             }
             else
                 return attributes;