Remove volatile on field in ConcurrentDictionary.Tables (#36976)

stephentoub · web-flow · commit d0e22812943f · 2020-05-25T07:18:01.000-04:00
diff --git a/src/libraries/System.Collections.Concurrent/src/System/Collections/Concurrent/ConcurrentDictionary.cs b/src/libraries/System.Collections.Concurrent/src/System/Collections/Concurrent/ConcurrentDictionary.cs
@@ -44,7 +44,7 @@ private sealed class Tables
         {
             internal readonly Node[] _buckets; // A singly-linked list for each bucket.
             internal readonly object[] _locks; // A set of locks, each guarding a section of the table.
-            internal volatile int[] _countPerLock; // The number of elements guarded by each lock.
+            internal readonly int[] _countPerLock; // The number of elements guarded by each lock.
 
             internal Tables(Node[] buckets, object[] locks, int[] countPerLock)
             {

Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ private sealed class Tables`
`44`	`44`	`{`
`45`	`45`	`internal readonly Node[] _buckets; // A singly-linked list for each bucket.`
`46`	`46`	`internal readonly object[] _locks; // A set of locks, each guarding a section of the table.`
`47`		`- internal volatile int[] _countPerLock; // The number of elements guarded by each lock.`
	`47`	`+ internal readonly int[] _countPerLock; // The number of elements guarded by each lock.`
`48`	`48`
`49`	`49`	`internal Tables(Node[] buckets, object[] locks, int[] countPerLock)`
`50`	`50`	`{`