[jvm] Fix off-by-one in multi cache

Raku · Dec 9, 2017 · 7eaebf5 · 7eaebf5
1 parent 47db26d
commit 7eaebf5
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/vm/jvm/runtime/org/perl6/nqp/sixmodel/reprs/MultiCacheInstance.java b/src/vm/jvm/runtime/org/perl6/nqp/sixmodel/reprs/MultiCacheInstance.java
@@ -96,9 +96,12 @@ public void add(CallCaptureInstance capture, SixModelObject result, ThreadContex
         if (numArgs >= MD_CACHE_MAX_ARITY)
             return;
 
+        /* The zero arity case was handled above.
+         * At index 0 we have the cache for arity 1 */
+        ArityCache ac = this.arityCaches[numArgs - 1];
+
         /* If the cache is saturated, don't do anything (we could instead do a random
          * replacement). */
-        ArityCache ac = this.arityCaches[numArgs];
         if (ac != null && ac.numEntries == MD_CACHE_MAX_ENTRIES)
             return;