commit 3a22666ba7279c0975d9c4cc8d90469132287ad4
author Stephen Adams <sra@google.com> Mon Jul 02 22:28:26 2018 +0000
committer commit-bot@chromium.org <commit-bot@chromium.org> Mon Jul 02 22:28:26 2018 +0000
tree cdf6e7c53cc5b6543e4a67a6038e795e6fb170f0
parent adb4b4af2cdef255c850b565bdc70c2c92d6b79c

Clearing a LinkedHashMap should not do big allocations

dart2js has some phases that clear() data structures at the end of the phase.
It was not our expectation that another 1M element list would be allocated at that time.

Change-Id: I8bbaf5ec7e9dbbde96b5e3a113f110af9e320912
Reviewed-on: https://dart-review.googlesource.com/63481
Reviewed-by: Ryan Macnak <rmacnak@google.com>
Commit-Queue: Stephen Adams <sra@google.com>

runtime/lib/compact_hash.dart

diff --git a/runtime/lib/compact_hash.dart b/runtime/lib/compact_hash.dart
index 21179eb..e9df2c8 100644
--- a/runtime/lib/compact_hash.dart
+++ b/runtime/lib/compact_hash.dart
@@ -169,7 +169,7 @@
 
   void clear() {
     if (!isEmpty) {
-      _init(_data.length, _hashMask, null, 0);
+      _init(_HashBase._INITIAL_INDEX_SIZE, _hashMask, null, 0);
     }
   }
 
@@ -493,7 +493,7 @@
 
   void clear() {
     if (!isEmpty) {
-      _init(_index.length, _hashMask, null, 0);
+      _init(_HashBase._INITIAL_INDEX_SIZE, _hashMask, null, 0);
     }
   }